Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0034977Openbravo ERPC. Securitypublic2017-01-17 10:142017-03-15 20:18
egoitz 
inigosanchez 
urgentminoralways
closedfixed 
5
 
3.0PR17Q2 
alostale
Core
Production - Confirmed Stable
2016-06-15
3.0PR16Q3
https://code.openbravo.com/erp/devel/pi/rev/463564b77a032485223cc070cf0aa66f6f0e03d8 [^]
No
0034977: A deactivated organization can not be activated again.
If you deactivate an organization you can not activate it again as the row is not ediable.
-Access on an application with F&B data
-Using admin role deactivate the España sur organization
-Logout and login with the same role.
-check that the España sur organization is not editable so you can not activate it
No tags attached.
depends on backport 00350643.0PR17Q1 closed inigosanchez A deactivated organization can not be activated again. 
depends on backport 00350653.0PR16Q4.3 closed inigosanchez A deactivated organization can not be activated again. 
depends on backport 00350663.0PR16Q3.6 closed inigosanchez A deactivated organization can not be activated again. 
caused by defect 0033166 closed alostale Role with access to one organization can edit all of them in Organization window 
Issue History
2017-01-17 10:14egoitzNew Issue
2017-01-17 10:14egoitzAssigned To => platform
2017-01-17 10:14egoitzModules => Core
2017-01-17 10:14egoitzTriggers an Emergency Pack => No
2017-01-18 14:34eugeniIssue Monitored: eugeni
2017-01-20 08:49alostalePrioritynormal => urgent
2017-01-20 08:49alostaleStatusnew => acknowledged
2017-01-20 08:49alostaleSummaryAn deactivated organization can not be activated again. => A deactivated organization can not be activated again.
2017-01-20 08:52alostaleStatusacknowledged => scheduled
2017-01-20 08:52alostaleAssigned Toplatform => inigosanchez
2017-01-23 13:15maiteResolution time => 1486249200
2017-01-25 08:37caristuRelationship addedcaused by 0033166
2017-01-25 09:16caristuRegression level => Production - Confirmed Stable
2017-01-25 09:16caristuRegression date => 2016-06-15
2017-01-25 09:16caristuRegression introduced in release => 3.0PR16Q3
2017-01-25 09:16caristuRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/463564b77a032485223cc070cf0aa66f6f0e03d8 [^]
2017-01-25 09:16caristuSeveritymajor => minor
2017-01-27 01:01inigosanchezStatusscheduled => acknowledged
2017-01-27 01:01inigosanchezStatusacknowledged => scheduled
2017-01-27 01:43inigosanchezReview Assigned To => caristu
2017-01-27 01:45hgbotCheckin
2017-01-27 01:45hgbotNote Added: 0093855
2017-01-27 01:45hgbotStatusscheduled => resolved
2017-01-27 01:45hgbotResolutionopen => fixed
2017-01-27 01:45hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/356f374645a18c17873328ef74e9be6bca414fcc [^]
2017-01-27 11:52inigosanchezReview Assigned Tocaristu => alostale
2017-01-27 11:52inigosanchezNote Added: 0093872
2017-02-01 16:03alostaleNote Added: 0093991
2017-02-01 16:03alostaleStatusresolved => new
2017-02-01 16:03alostaleResolutionfixed => open
2017-02-01 16:14inigosanchezStatusnew => scheduled
2017-02-01 19:14hgbotCheckin
2017-02-01 19:14hgbotNote Added: 0093998
2017-02-01 19:15inigosanchezStatusscheduled => resolved
2017-02-01 19:15inigosanchezResolutionopen => fixed
2017-02-03 07:13alostaleNote Added: 0094022
2017-02-03 07:13alostaleStatusresolved => closed
2017-02-03 07:13alostaleFixed in Version => 3.0PR17Q2
2017-02-27 16:39inigosanchezNote Added: 0094644
2017-03-15 20:18hudsonbotCheckin
2017-03-15 20:18hudsonbotNote Added: 0095123
2017-03-15 20:18hudsonbotCheckin
2017-03-15 20:18hudsonbotNote Added: 0095151

Notes
(0093855)
hgbot   
2017-01-27 01:45   
Repository: erp/devel/pi
Changeset: 356f374645a18c17873328ef74e9be6bca414fcc
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Fri Jan 27 01:44:41 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/356f374645a18c17873328ef74e9be6bca414fcc [^]

Fixed issue 34977: A deactivated organization can not be activated again.

When an organization was disabled you could not activate it again as the row was not ediable. This
problem was introduced in a fix related with the code of addWritableAttribute() method of
DefaultJsonDataService class. Before this fix, It was managing the reported case properly (Activate
again a deactivated organization) because this method was handling the particular case of Organizations.

Now is taking into account this particular case (deactivate organizations) in order to manages this
particular case properly. Now it is possible to activate again a deactivate organization.

---
M modules/org.openbravo.service.json/src/org/openbravo/service/json/DefaultJsonDataService.java
M src/org/openbravo/dal/core/OBContext.java
M src/org/openbravo/dal/security/SecurityChecker.java
---
(0093872)
inigosanchez   
2017-01-27 11:52   
TestLink > 3.0 Test Suite > Regression test cases > Test cases for 3.0PR16Q3:
> 34977 A deactivated organization should be activated again
> 34977 Role with access to one organization can't edit the rest of organizations
(0093991)
alostale   
2017-02-01 16:03   
Tested -> working fine
Code review -> some minor issues (don't require to be backported) https://docs.google.com/spreadsheets/d/18l2biKWeG6iGDIPqO6VuYQ_TtP_6YxyQU4KSrPvQlJ8/edit#gid=0 [^]
(0093998)
hgbot   
2017-02-01 19:14   
Repository: erp/devel/pi
Changeset: 0ebde0551a1ffd24a8bbe1345ad00d895039e3b6
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Wed Feb 01 19:13:34 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/0ebde0551a1ffd24a8bbe1345ad00d895039e3b6 [^]

Related with issue 34977: Codereview improvements

Some changes related with improving performance and legibility.

---
M modules/org.openbravo.service.json/src/org/openbravo/service/json/DefaultJsonDataService.java
M src/org/openbravo/dal/core/OBContext.java
---
(0094022)
alostale   
2017-02-03 07:13   
code reviewed and tested
(0094644)
inigosanchez   
2017-02-27 16:39   
Added automatic test in selenium (pi-smoke-platform): https://code.openbravo.com/tools/automation/pi-smoke-platform/rev/0745d9721db5 [^]
(0095123)
hudsonbot   
2017-03-15 20:18   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/54e102bef53e [^]
Maturity status: Test
(0095151)
hudsonbot   
2017-03-15 20:18   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/54e102bef53e [^]
Maturity status: Test