[Yanel-commits] rev 49840 - public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet

[michi at wyona.com]

Author: michi
Date: 2010-05-25 13:07:45 +0200 (Tue, 25 May 2010)
New Revision: 49840

Modified:
   public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
Log:
comments added re suppressing toolbar

Modified: public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
===================================================================
--- public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2010-05-25 09:08:38 UTC (rev 49839)
+++ public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2010-05-25 11:07:45 UTC (rev 49840)
@@ -1927,9 +1927,10 @@
             // Possibly embed toolbar:
             // TODO: Check if user is authorized to actually see toolbar (Current flaw: Enabled Toolbar, Login, Toolbar is enabled, Logout, Toolbar is still visible!)
             if (yanelUI.isToolbarEnabled(request)) {
+                // TODO: if ("suppress".equals(res.getResConfiguration("yanel.toolbar"))) {
                 if (mimeType != null && mimeType.indexOf("html") > 0) {
-                    // TODO: What about other query strings or frames or TinyMCE?
-                    if (request.getParameter(YANEL_RESOURCE_USECASE) == null) {
+                    // TODO: What about other query strings or frames or TinyMCE (e.g. link.htm)?
+                    if (request.getParameter(YANEL_RESOURCE_USECASE) == null) { // INFO: In the case of a yanel resource usecase do NOT add the toolbar
                         if (toolbarMasterSwitch.equals("on")) {
                             OutputStream os = response.getOutputStream();
                             try {