if no allowed roles is provided it checks only if a user is logged

geosolutions-it · Oct 13, 2016 · b10424a · b10424a
1 parent d513656
commit b10424a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/web/client/plugins/SaveAs.jsx b/web/client/plugins/SaveAs.jsx
@@ -174,8 +174,12 @@ module.exports = {
             text: <Message msgId="saveAs"/>,
             icon: <Glyphicon glyph="floppy-open"/>,
             action: editMap.bind(null, {}),
-            selector: (state) => (
-                indexOf(state && state.controls && state.controls.saveAs && state.controls.saveAs.allowedRoles, state && state.security && state.security.user && state.security.user.role) !== -1 ? {} : { style: {display: "none"} })
+            selector: (state) => {
+                if (state && state.controls && state.controls.saveAs && state.controls.saveAs.allowedRoles) {
+                    return indexOf(state.controls.saveAs.allowedRoles, state && state.security && state.security.user && state.security.user.role) !== -1 ? {} : { style: {display: "none"} };
+                }
+                return state && state.security && state.security.user ? {} : { style: {display: "none"} };
+            }
         }
     }))
 };
diff --git a/web/client/product/appConfig.js b/web/client/product/appConfig.js
@@ -46,9 +46,6 @@ module.exports = {
                 drawer: {
                     enabled: false,
                     menu: "1"
-                },
-                saveAs: {
-                    allowedRoles: ["ADMIN", "USER"]
                 }
             }
         },