[SLING-11311] Improve org-apache-sling-jcr-resource

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: JCR Resource 3.2.0
• Fix Version/s: JCR Resource 3.2.2
• Component/s: JCR
• Assignee: Angela Schreiber
• Reporter: Angela Schreiber
• Created: 11/May/22 08:57
• Updated: 28/Sep/22 09:42
• Resolved: 21/Jul/22 09:38

Description

while working on SLING-10011 i noticed a considerable amount of issues reported by sonar, which are trivial to fix and would improve overall code quality.

apart from sonar findings we should fix

• inconsistent formatting
• unused imports
• missing nullable/notnull annotations
• complexity

Issue Links

• relates to: SLING-10011 Use JackrabbitSession.getParentOrNull when resolving parent JCR node in JcrResourceProvider#getParent

Sub-Tasks

• Fix inconsistent formatting: Closed
• Fix nullable/notnull annotations: Closed
• Fix redundant constructs: Closed
• Fix javadoc problems: Closed
• JcrListenerBaseConfig references wrong class in logger: Closed
• Sonar findings (excluding complexity and nullable/notnull annotations): Closed
• Potential NPE as ResolveContext.getProviderState() is nullable: Closed
• Add missing notnull/nullable annotations: Closed
• Reduce complexity: Closed
• Simplify JcrPropertyMapCacheEntry.convertToType: Closed