Considering that URIs like 'mailto:j...@example.com', '?name=val'
or '#fragment' do not relate to a resource and thinking about it
a bit more, I would now propose to change the name of
ResourceUri/ResourceUriBuilder to SlingUri/SlingUriBuilder
(see also [1]) - objections?

-Georg

[1] https://github.com/apache/sling-org-apache-sling-api/pull/23#discussion_r491726353

On 2020-09-18 11:14, Georg Henzler wrote:
Hi all,

to not have too many changes in one issue and PR to review,
I created SLING-9745 and PR [1] for the ResourceUri and
its builder. The unit test coverage is good (~85%) and also
covers the tests from SlingRequestPathInfoTest [2] to
ensure compatibility. Timely feedback would be appreciated
as I would like to build upon this on Mon (with the next PR).

-Georg

[1]
https://github.com/apache/sling-org-apache-sling-api/pull/23
[2]
https://github.com/apache/sling-org-apache-sling-engine/blob/43673bddb4a2e79ef9bba813ec86a7faa89e7649/src/test/java/org/apache/sling/engine/impl/request/SlingRequestPathInfoTest.java#L28

Reply via email to