static private final String VIEW_STATE_DISPLAY_RESULT = "VIEW_STATE_DISPLAY_RESULT";
- static private final String VIEW_STATE_DISPLAY_SERVICE = "VIEW_STATE_DISPLAY_SERVICE";
+// static private final String VIEW_STATE_DISPLAY_SERVICE = "VIEW_STATE_DISPLAY_SERVICE";
+
+ private static final String XSL_FILE = "/WEB-INF/lib/xsl/portlet.xsl";
+
Non mi piace molto avere il path dell'XSL in EnginFramePortlet. Perchè non lo abbiamo hardcoded nel transformer?
AA: il motivo è per genrializzare generateHTML, pasadno il path all'XSL chiunque può chiamarlo
private String sdfURL; // URL to then EnginFrame Service Definition File
private String baseURL;
+ private String portletXSL;
Come sopra..
AA: Come sopra..
- String html = PortletXSLTransformer.generateHTML(this, request, response, baseURL, service.getId(), result.value);
+ log("submitServiceWithNoOptions(result):\n"+result.value);
+ String html = PortletXSLTransformer.generateHTML(request, response, baseURL, Utils.extractURI(result.value), false, portletXSL, result.value);
Non è meglio passare "this" e "serviceId" e calcolare il path all'XSL e fare extractURI nel tranformer? Qualcosa in contrario se modifico il
tutto per fare così?
AA: OK, secondo me dovresti mettere dei getters nella Portlet, e.g., getBaseURL, getXSL, ecc.
Oppure possiamo creare una classe, e.g. TransformationParams, che contiene i dati della trasformazione
+
+ protected String getTitle(RenderRequest request) {
+ if(cache != null) {
Se non dico cazzate, cache è sempre != null perchè è allocata nel costruttore.
AA: Non so perché, ma a un certo punto mi aveva dato NPE
+ Service service = cache.getService(request);
+ if(service != null) {
+ return service.getName();
+ }
+ }
+ return super.getTitle(request);
+ }
Vuoi usare il nome del servizio come titolo della finestra?
AA: non vi piace come idea?
+}
\ No newline at end of file