Skip to content

Removed NOP condition from ContentService#771

Merged
konradoboza merged 1 commit into
5.0from
cleanup
Jul 2, 2026
Merged

Removed NOP condition from ContentService#771
konradoboza merged 1 commit into
5.0from
cleanup

Conversation

@mnocon

@mnocon mnocon commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Removing a leftover comment (?) I've noticed when looking at code related to #543 , and trying to understand fieldtype renaming.

@sonarqubecloud

sonarqubecloud Bot commented Jul 2, 2026

Copy link
Copy Markdown

@konradoboza konradoboza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great catch!

@bnowak bnowak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😅

@micszo micszo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Fire at will!

@alongosz alongosz changed the title Removed NOP condition Removed NOP condition from ContentService Jul 2, 2026
@konradoboza konradoboza merged commit bdde0e7 into 5.0 Jul 2, 2026
34 checks passed
@konradoboza konradoboza deleted the cleanup branch July 2, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants