Skip to content

Conversation

@tbialcz
Copy link
Contributor

@tbialcz tbialcz commented Jul 10, 2025

🎫 Issue IBX-10262

Description:

This PR renames and refactors the existing CustomTagsValidator class to CustomTemplateValidator, expanding its responsibility to validate both:

  • Custom Tags β€” including required attributes validation
  • Custom Styles β€” skipping attribute checks, as styles don't define any

@tbialcz tbialcz requested a review from a team July 10, 2025 14:09
@ezrobot ezrobot requested review from Steveb-p, ViniTou, barw4, ciastektk, konradoboza, mikadamczyk and wiewiurdp and removed request for a team July 10, 2025 14:09
@sonarqubecloud
Copy link

@tbialcz tbialcz requested a review from adamwojs July 10, 2025 14:25
@adamwojs
Copy link
Member

It works.

image

* The Validator checks if the given XML reflects proper Custom Tags configuration,
* mostly existence of specific Custom Tag and its required attributes.
*/
class CustomTagsValidator implements ValidatorInterface
Copy link
Member

Choose a reason for hiding this comment

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

Please make sure this change is documented. This is BC break.

Copy link
Contributor

Choose a reason for hiding this comment

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

@mnocon requires your attention 😊

@adamwojs adamwojs merged commit 372c366 into 4.6 Jul 10, 2025
22 checks passed
@adamwojs adamwojs deleted the ibx_10262 branch July 10, 2025 16:29
@mnocon mnocon added the Doc needed The changes require some documentation label Jul 11, 2025
@mnocon mnocon removed the Doc needed The changes require some documentation label Aug 5, 2025
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.

6 participants