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.

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

TextHandler should extends FaceletHandler #2008

Open
pizzi80 opened this issue Feb 10, 2025 · 3 comments
Open

TextHandler should extends FaceletHandler #2008

pizzi80 opened this issue Feb 10, 2025 · 3 comments
Labels
EE12 myfaces-implemented API issue implemented by MyFaces
Milestone

Comments

@pizzi80
Copy link

pizzi80 commented Feb 10, 2025

the documentation says:

An interface that allows other code to identify FaceletHandlers that may provide text (String) content.

@pizzi80 pizzi80 changed the title TextHandler should extends FaceletHandler [5.0] TextHandler should extends FaceletHandler Feb 18, 2025
@tandraschko
Copy link

not sure about that, would need to check history in detail

WDYT @BalusC ?

@pizzi80
Copy link
Author

pizzi80 commented Feb 24, 2025

just a little info:

I've made this change in my fork many weeks ago

and thanks to this I've improved the mojarra TagHandlerImpl class

reducing the method findNextByType(Class<T> type)
to one static utility method

@BalusC
Copy link
Member

BalusC commented Feb 24, 2025

Makes sense. Javadoc literally says TextHandler is a FaceletHandler.

@tandraschko tandraschko added this to the 5.0 milestone Feb 25, 2025
@tandraschko tandraschko changed the title [5.0] TextHandler should extends FaceletHandler TextHandler should extends FaceletHandler Mar 1, 2025
@tandraschko tandraschko added the myfaces-implemented API issue implemented by MyFaces label Mar 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EE12 myfaces-implemented API issue implemented by MyFaces
Projects
None yet
Development

No branches or pull requests

4 participants