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.

Sign up for GitHub

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

Jump to bottom

[core] Cleanups in XPath area #4309

Closed
3 tasks done
Tracked by #3898
adangel opened this issue Jan 11, 2023 · 0 comments · Fixed by #4804
Closed
3 tasks done
Tracked by #3898

[core] Cleanups in XPath area #4309

adangel opened this issue Jan 11, 2023 · 0 comments · Fixed by #4804
Assignees
@adangel
Labels
in:xpath Relating to xpath support at large, eg Jaxen / Saxon, custom functions, attribute resolution
Milestone
7.0.0

Comments

@adangel
Copy link
Member

adangel commented Jan 11, 2023 •
edited

  • Move XPathRule to correct package net.sourceforge.pmd.lang.rule.xpath. For compatibility, we could keep a @DeprecatedUntilPmd700 subclass
    in place. Depends on the final ruleset xml format.
  • [deprecations] The property version of XPathRule is deprecated and should be removed.
  • [api] AbstractXPathFunctionDef/XPathHandler -> uses saxon api and leaks this into pmd api for language implementations
@adangel adangel added the in:xpath Relating to xpath support at large, eg Jaxen / Saxon, custom functions, attribute resolution label Jan 11, 2023
@adangel adangel added this to the 7.0.0 milestone Jan 11, 2023
@adangel adangel mentioned this issue Jan 11, 2023
Closed
14 tasks
@adangel adangel mentioned this issue Jan 23, 2023
Closed
55 tasks
@adangel adangel self-assigned this Jan 19, 2024
adangel added a commit to adangel/pmd that referenced this issue Jan 20, 2024
@adangel
[core] Move XPathRule into subpackage lang.rule.xpath (pmd#4309)
7c29f16
adangel added a commit to adangel/pmd that referenced this issue Jan 20, 2024
@adangel
[core] XPathRule - remove version property (pmd#4309)
049f2c7
@adangel adangel mentioned this issue Jan 20, 2024
Merged
4 tasks
adangel added a commit to adangel/pmd that referenced this issue Jan 25, 2024
@adangel
[doc] Update release notes (pmd#4309)
f5c4aad
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adangel adangel

Labels
in:xpath Relating to xpath support at large, eg Jaxen / Saxon, custom functions, attribute resolution
Projects
None yet
Milestone
7.0.0
Development

Successfully merging a pull request may close this issue.

[core] XPath cleanups adangel/pmd
1 participant
@adangel