Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tags 3.0: rename java.sun.com / xmlns.jcp.org taglib URIs to new jakarta.tags.* URNs #144

Closed
BalusC opened this issue Feb 21, 2021 · 4 comments
Closed

Tags 3.0: rename java.sun.com / xmlns.jcp.org taglib URIs to new jakarta.tags.* URNs #144

BalusC opened this issue Feb 21, 2021 · 4 comments
Assignees
@volosied
Labels
JakartaEE10

Comments

@BalusC
Copy link
Member

BalusC commented Feb 21, 2021 •
edited

In line with jakartaee/faces#1553

Proposal: use URNs instead of URLs as URLs are more prone to confuse people because they expect them to be available as actual web resources along with XSDs.

Also: remove "JSP" from new URIs as Jakarta Tags API allows it being reused in other view technologies than Jakarta Pages, such as Facelets.

E.g.

Rename http://xmlns.jcp.org/jsp/jstl/core to jakarta.tags.core
Rename http://xmlns.jcp.org/jsp/jstl/fmt to jakarta.tags.fmt

Then in JSP

<%@taglib prefix="c" uri="jakarta.tags.core" %>
<%@taglib prefix="fmt" uri="jakarta.tags.fmt" %>
<html>
    ...
</html>

Or in Facelets

<html
    xmlns="http://www.w3.org/1999/xhtml"
    xmlns:c="jakarta.tags.core"
    xmlns:fmt="jakarta.tags.fmt"
>
    ...
</html>

Thoughts?
@GedMarc GedMarc mentioned this issue Mar 22, 2021
Closed
3 tasks
@BalusC
Copy link
Member Author

BalusC commented Mar 27, 2021 •
edited

FYI: I've proactively implemented support for jakarta.tags.core and jakarta.tags.functions for Faces 4.0: jakartaee/faces#1553 This is because Faces uses internally the Facelets view technology which has its own implementation for the "core" and "functions" libraries of the Jakarta Tags API.

Old xmlns.jcp.org and even java.sun.com namespace URIs will keep working fine in Faces 4.0.

If you want to rename the abovementioned jakarta.tags.* to something else, let me know, then I'll sync up in Faces.

@BalusC BalusC mentioned this issue Apr 14, 2021
Closed
@pnicolucci
Copy link
Contributor

@arjantijms @BalusC I'm working on the plan review for Jakarta EE10 for this project. See: https://www.eclipse.org/lists/jstl-dev/msg00063.html

Do we think this change requires a Tags 3.0 release or a Tags 2.1? I see the title here refers to Tags 3.0 but just wanted to double-check that a change of this nature would be a change that warrants a major release.

@pnicolucci pnicolucci mentioned this issue Oct 21, 2021
Closed
@pnicolucci
Copy link
Contributor

@volosied we'll also want to ensure that the tlddoc looks correct once these changes are made.

@pnicolucci
Copy link
Contributor

Also, we'll want to look at what TCK tests are available for testing these new changes and update where necessary. We can discuss that as we make progress with this issue.

This was referenced Nov 10, 2021
Merged
Closed
@alwin-joseph alwin-joseph mentioned this issue Mar 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@volosied volosied

Labels
JakartaEE10
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@BalusC @volosied @pnicolucci