details:   https://code.openbravo.com/erp/devel/pi/rev/af4557ab55b2
changeset: 16620:af4557ab55b2
user:      Guillermo Álvarez de Eulate <guillermo.alvarez <at> openbravo.com>
date:      Thu May 24 18:00:18 2012 +0200
summary:   Fixed issue 20544: FIC SETSESSION shouldn't set the session with 
null values

diffstat:

 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
 |  21 +++++----
 1 files changed, 12 insertions(+), 9 deletions(-)

diffs (31 lines):

diff -r 28a4b99b5265 -r af4557ab55b2 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
--- 
a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
     Thu May 24 17:55:35 2012 +0200
+++ 
b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
     Thu May 24 18:00:18 2012 +0200
@@ -1483,15 +1483,18 @@
         // of the attachments section are created
         if (savingNewRecord) {
           this.attachmentsSection.fillAttachments(null);
-          // We also do a call to the FIC on SETSESSION mode to set the 
session variables
-          // to fix issue 18453
-          sessionProperties = this.view.getContextInfo(true, true, false, 
true);
-          
OB.RemoteCallManager.call('org.openbravo.client.application.window.FormInitializationComponent',
 sessionProperties, {
-            MODE: 'SETSESSION',
-            TAB_ID: this.view.tabId,
-            PARENT_ID: this.view.getParentId(),
-            ROW_ID: this.values.id
-          }, null);
+          //Issue 20544 -> The session shouldn't be setted with a null value
+          if (this.values.id) {
+            // We also do a call to the FIC on SETSESSION mode to set the 
session variables
+            // to fix issue 18453
+            sessionProperties = this.view.getContextInfo(true, true, false, 
true);
+            
OB.RemoteCallManager.call('org.openbravo.client.application.window.FormInitializationComponent',
 sessionProperties, {
+              MODE: 'SETSESSION',
+              TAB_ID: this.view.tabId,
+              PARENT_ID: this.view.getParentId(),
+              ROW_ID: this.values.id
+            }, null);
+          }
         }
 
       } else if (status === isc.RPCResponse.STATUS_VALIDATION_ERROR && 
resp.errors) {

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to