[jira] [Issue Comment Deleted] (JCRVLT-155) JcrPackage.isInstalled does not return false in case the package has been overwritten

["Konrad Windszus (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/JCRVLT-155?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Konrad Windszus updated JCRVLT-155:
-----------------------------------
    Comment: was deleted

(was: What about a more sophisticated approach which would compare the created date of the old and the new package. If that differs the state should not be transferred from the old to the new package. That way this would work for both: modified SNAPSHOT versions and modified release versions (having the same version string).)

> JcrPackage.isInstalled does not return false in case the package has been overwritten
> -------------------------------------------------------------------------------------
>
>                 Key: JCRVLT-155
>                 URL: https://issues.apache.org/jira/browse/JCRVLT-155
>             Project: Jackrabbit FileVault
>          Issue Type: Bug
>          Components: Packaging
>    Affects Versions: 3.1.28
>            Reporter: Konrad Windszus
>
> Although the javadoc of {{o.a.j.v.packaging.JcrPackage#isInstalled()}} is stating
> bq. Checks if this package is installed. Note: the default implementation only checks the JcrPackageDefinition.getLastUnpacked() date. If the package is replaced since it was installed. this method will return false.
> this method is returning {{true}} in case the package has been reuploaded in between (with the same group, package name and version). This is violating the contract of this method.
> Please either fix implementation or javadoc description.
> That issue already lead to downstream issues in the {{com.adobe.granite.installer.factory.packages.impl.PackageTransformer}} which therefore only installs a package once.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)