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

Feature/#59 bootsfaces renderers #29

Merged
merged 3 commits into from
Dec 1, 2018

Conversation

woutbr
Copy link
Owner

@woutbr woutbr commented Dec 1, 2018

Dit lost het probleem op dat soms Glassfish opstart en een witte pagina toont met enkel tekst.

Dit komt omdat soms Glassfish de classpath niet scant bij het opstarten en dus de bootfaces-jar niet gebruikt. Onze bootsfaces tags worden dan niet gerendered.

De oplossing is zelf al de bootsfaces elementen te registreren. InitBootsfacesBean.initializeBootsfacesRenderers wordt dan bij elke startup gecalled en zal alle componenten onder net.bootsfaces.component.ComponentsEnum toevoegen.

De oplossing heb ik hier gevonden: #TheCoder4eu/BootsFaces-OSP#732
Samengevat op deze blog: beyondjava.net/running-bootsfaces-on-glassfish-or-payara

PS: Bij mij kan het niet alle componenten registreren. Ik krijg nog als boodschap: Init Bootsfaces: registered renderers/total components : 68/78

woutbr and others added 3 commits November 24, 2018 13:01
Methode getDefaultRenderKit gekopieerd uit org.primefaces.mobile.renderkit.MobileRenderKit
@ManagedBean veranderd naar @nAmed omwille van nieuwere CDI.
Methode startup toegevoegd welke de startup van de application observed.
@woutbr woutbr added the enhancement New feature or request label Dec 1, 2018
@woutbr woutbr requested a review from vcremers December 1, 2018 10:30
@vcremers vcremers merged commit c039751 into develop Dec 1, 2018
@vcremers vcremers deleted the feature/#59_BootsfacesRenderers branch December 1, 2018 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants