kwin commented on code in PR #49:
URL:
https://github.com/apache/sling-org-apache-sling-jcr-resource/pull/49#discussion_r2376201921
##########
src/main/java/org/apache/sling/jcr/resource/internal/JcrValueMap.java:
##########
@@ -300,10 +330,17 @@ public String getPath() {
try {
final String key = escapeKeyName(name);
Property property = NodeUtil.getPropertyOrNull(node,key);
- if (property == null &&
name.equals(ResourceResolver.PROPERTY_RESOURCE_TYPE)) {
- // special handling for the resource type property which
according to the API must always be exposed via property sling:resourceType
- // use value of jcr:primaryType if sling:resourceType is not
set
- property = NodeUtil.getPropertyOrNull(node,
JcrConstants.JCR_PRIMARYTYPE);
+ if (property == null) {
+ if (name.equals(ResourceResolver.PROPERTY_RESOURCE_TYPE)) {
+ // special handling for the resource type property which
according to the API must always be exposed via property sling:resourceType
+ // use value of jcr:primaryType if sling:resourceType is
not set
+ JcrPropertyMapCacheEntry entry =
read(JcrConstants.JCR_PRIMARYTYPE);
Review Comment:
It is with
https://github.com/apache/sling-org-apache-sling-jcr-resource/pull/49/files/45a0c9fe53c01290e576915792f12f7466762816.
This is only the fallback.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
