[ 
https://issues.apache.org/jira/browse/JCRVLT-526?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17954585#comment-17954585
 ] 

Sagar Miglani commented on JCRVLT-526:
--------------------------------------

[~kwin] [~reschke] 
Previously, date formats such as {{"2025-05-26T10:05"}} appeared to be working. 
It seems that this change has affected this behavior.

Could you kindly clarify whether this date format was officially intended to be 
supported?

I am getting the following exception:
{code:xml}
28.05.2025 18:12:34.785 *ERROR* 
org.apache.jackrabbit.vault.packaging.impl.PackagePropertiesImpl Error while 
converting date property
java.time.format.DateTimeParseException: Text '2025-05-28T18:05' could not be 
parsed at index 16
    at 
java.base/java.time.format.DateTimeFormatter.parseResolved0(DateTimeFormatter.java:2052)
    at 
java.base/java.time.format.DateTimeFormatter.parse(DateTimeFormatter.java:1954)
    at java.base/java.time.ZonedDateTime.parse(ZonedDateTime.java:600)
    at 
org.apache.jackrabbit.vault.packaging.impl.PackagePropertiesImpl.getDateProperty(PackagePropertiesImpl.java:249)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.impl.PackagePropertiesImpl.getCreated(PackagePropertiesImpl.java:103)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.impl.JcrPackageDefinitionImpl.writeProperties(JcrPackageDefinitionImpl.java:398)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.impl.JcrPackageDefinitionImpl.unwrap(JcrPackageDefinitionImpl.java:239)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.registry.impl.JcrPackageRegistry.createNew(JcrPackageRegistry.java:624)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.registry.impl.JcrPackageRegistry.upload(JcrPackageRegistry.java:489)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.registry.impl.JcrPackageRegistry.upload(JcrPackageRegistry.java:422)
 [org.apache.jackrabbit.vault:3.8.0]
    at 
org.apache.jackrabbit.vault.packaging.impl.JcrPackageManagerImpl.upload(JcrPackageManagerImpl.java:250)
 [org.apache.jackrabbit.vault:3.8.0]
{code}

> PackagePropertiesImpl doesn't support date format with timezone format +-hhmm
> -----------------------------------------------------------------------------
>
>                 Key: JCRVLT-526
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-526
>             Project: Jackrabbit FileVault
>          Issue Type: Bug
>         Environment: Sling-Starter 12-SNAPSHOT with JDK 11 on MacOS
>            Reporter: Hans-Peter Stoerr
>            Assignee: Konrad Windszus
>            Priority: Minor
>             Fix For: 3.5.4
>
>
> When packages e.g. a creation date with a timezone format +- hhmm or +-hh are 
> entered into the FSPackageRegistry, the date cannot be read by 
> org.apache.jackrabbit.vault.packaging.impl.PackagePropertiesImpl.getCreated() 
> : it returns null since it internally uses 
> org.apache.jackrabbit.util.ISO8601.parse to parse the date, which doesn't 
> support that (compare JCR-4267). This is a problem, since 
> com.day.jcr.vault:content-package-maven-plugin (I tried versions 0.0.12 and 
> 0.5.4) write package properties using such a format - e.g. 
> 2021-05-26T15:12:21.673+0200 , while org.apache.jackrabbit.util.ISO8601.parse 
> just supports a format like 2021-05-26T15:12:21.673+02:00 . So right now one 
> has to use unpleasant workarounds like using 
> packageProps.getProperty(PackageProperties.NAME_CREATED) and parse that. I 
> suggest changing PackagePropertiesImpl.getDateProperty to support this 
> format, instead.
> (Compare also this comment , where that was discussed.)



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to