Hi Antonio,
> Here some related notes:
>
> - the package version is kept the same
> org.apache.sling.jcr.resource;version=2.1 in both bundle version
> - the containsKey method is not part of any API (AFAIK) but is part of the
> org.apache.sling.jcr.resource package and is public
It's part of the Map API and ValueMap extends from Map.
> - the exception is thrown from the read method that now is public and
> declares the runtime exception in the Java Doc (before the method was package
> protected)
Hmm, as far as I see read() is still package protected.
> - previously the Exception was "swallowed" in the read method
So what did containsKey("") return? Just false?
>
> Again probably is not an issue at all but I am still "curious" to know WDYT :)
I think you're bringing up an important point. We shouldn't change the
behaviour unless the version change is a major version change. So,
even if we would have exported the api package as 2.2, the behaviour
should still be the same. The only exception would be if the previous
behaviour is a bug :)
So, in short, if containsKey("") worked before correctly (returning
false), then it should work the same now.
Regards
Carsten
>
> Regards
>
> Antonio
--
Carsten Ziegeler
[email protected]
