details:   https://code.openbravo.com/erp/devel/pi/rev/fd863146a4f4
changeset: 28684:fd863146a4f4
user:      Asier Lostalé <asier.lostale <at> openbravo.com>
date:      Fri Feb 26 08:57:54 2016 +0100
summary:   fixed bug 32354: field edit access doesn't allow to save if tab 
contains audit

  For tabs that explicitly define their audit (updated/updated by) fields, roles
  with access to edit only some fields couldn't save unless in those fields 
audit
  were also included.

  Now audit fields are not checked for this validation as they are always 
updated
  on any allowed modification.

diffstat:

 
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DefaultDataSourceService.java
 |  3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diffs (13 lines):

diff -r f2e7ebc3ea7f -r fd863146a4f4 
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DefaultDataSourceService.java
--- 
a/modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DefaultDataSourceService.java
       Fri Feb 26 08:20:01 2016 +0100
+++ 
b/modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DefaultDataSourceService.java
       Fri Feb 26 08:57:54 2016 +0100
@@ -254,6 +254,9 @@
       fieldQuery.setNamedParameter("roleId", roleId);
       for (Field f : fieldQuery.list()) {
         Property property = KernelUtils.getProperty(f);
+        if (property.isAuditInfo()) {
+          continue;
+        }
         String key = property.getName();
         if (data.has(key)) {
           String newValue = getValue(data, key);

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Openbravo-commits mailing list
Openbravo-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to