details:   https://code.openbravo.com/erp/devel/pi/rev/4cf5f2223453
changeset: 26227:4cf5f2223453
user:      Inigo Sanchez <inigo.sanchez <at> openbravo.com>
date:      Mon Mar 23 21:55:29 2015 +0100
summary:   Fixed issue 29066: Process definition should inherit permissions 
from window

The problem was that in some cases when a rol had access to a window, he
did not have access to the processes of the window. The problem was when a
proccess was referenced by a OBUISEL_Selector reference.

This problem happens when is controlled access process. In many cases the
windowId is null.

Now, this problem has been solved because windowId will have a value.

diffstat:

 
modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
 |  5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diffs (15 lines):

diff -r 86f26739f045 -r 4cf5f2223453 
modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
--- 
a/modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
      Mon Mar 23 17:27:10 2015 +0100
+++ 
b/modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
      Mon Mar 23 21:55:29 2015 +0100
@@ -938,6 +938,11 @@
       windowId: view.windowId,
       windowTitle: OB.I18N.getLabel('OBUISEL_AddNewRecord', [this.title])
     };
+    // Avoid that windowId is null. WindowId in params is necessary
+    // to check access process of OBUISEL_Selector Reference.
+    if (!params.windowId) {
+      params.windowId = view.standardWindow.windowId;
+    }
     if (additionalProcessProperties) {
       isc.addProperties(params, additionalProcessProperties);
     }

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to