Skip to content

Commit

Permalink
Fixed: Improve ViewHandler interface : missing screens in context (OF…
Browse files Browse the repository at this point in the history
…BIZ-13179)

After refactoring view handler to secure the context, the screenRenderer is missing for Screen Fop.

This is problematic if you want include some other screen on your fop template like :

${screens.render("component://mycompo/widget/MysScreens#IncludedScreen")}

Thanks Leila Mekika that detect this regression
  • Loading branch information
nmalin committed Dec 19, 2024
1 parent a3170e9 commit 637d01d
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,7 @@ public void render(String name, String page, String info, String contentType, St
// this is the object used to render forms from their definitions
screens.getContext().put("formStringRenderer", formStringRenderer);
screens.getContext().put("simpleEncoder", UtilCodec.getEncoder(modelTheme.getEncoder(getName())));
screens.getContext().put("screens", screens);
screens.render(page);
} catch (IOException | GeneralException | SAXException | ParserConfigurationException | TemplateException e) {
renderError("Problems with the response writer/output stream", e, "[Not Yet Rendered]", request, response, context);
Expand Down

0 comments on commit 637d01d

Please sign in to comment.