Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor "substr" calls in lib/private to improve code readability #39216

Merged

Conversation

shdehnavi
Copy link
Contributor

Summary

Replacing substr with str_starts_with and str_ends_with in the /lib/private namespace to improve code readability

Checklist

@shdehnavi shdehnavi changed the title Refactor "substr" calls to improve code readability Refactor "substr" calls in lib/private to improve code readability Jul 7, 2023
@solracsf solracsf added this to the Nextcloud 28 milestone Jul 9, 2023
@solracsf solracsf added 3. to review Waiting for reviews technical debt labels Jul 9, 2023
Copy link
Member

@fsamapoor fsamapoor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much much better. Thanks!
Other than the comment I left, looks good to me.

lib/private/DateTimeFormatter.php Outdated Show resolved Hide resolved
@shdehnavi shdehnavi requested review from artonge and removed request for ArtificialOwl July 20, 2023 08:41
Co-authored-by: Faraz Samapoor <[email protected]>
Signed-off-by: Hamid Dehnavi <[email protected]>
lib/private/DateTimeFormatter.php Outdated Show resolved Hide resolved
Co-authored-by: Louis <[email protected]>
Signed-off-by: Hamid Dehnavi <[email protected]>
lib/private/DateTimeFormatter.php Outdated Show resolved Hide resolved
lib/private/DateTimeFormatter.php Fixed Show fixed Hide fixed
lib/private/DateTimeFormatter.php Fixed Show fixed Hide fixed
Co-authored-by: Côme Chilliet <[email protected]>
Signed-off-by: Hamid Dehnavi <[email protected]>
@icewind1991 icewind1991 merged commit ef87ff1 into nextcloud:master Sep 21, 2023
33 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants