Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0028712Openbravo ERPA. Platformpublic2015-01-21 12:382015-03-02 20:31
caristu 
alostale 
urgentmajoralways
closedfixed 
5
pi 
3.0PR15Q23.0PR15Q2 
AugustoMauch
Core
No
0028712: Records selection is lost in pick and execute window under some circumnstances
Records selection is lost in pick and execute window under some circumstances. It happen when selecting records from different pages.
1) Install the attached module
2) Go to the [Sales Order] window. Click on the EM_Suptest_Proc which opens a P&E window.
3) Select a couple of records
4) Filter by docno column in order to display a record from another page. Select one of that records
5) Remove the filter
6) Select another record
7) Apply the same filter applied in 4). Notice that the selected record in 4) appears unselected
No tags attached.
related to defect 00284683.0PR15Q2 closed AugustoMauch Some values entered by a selection function are lost in P&E windows under some circumstances 
related to defect 00225223.0MP20 closed AugustoMauch In Pick&Execute windows a removed selection appears when sorting the columns 
related to defect 0028923 closed platform The multiple selector on the processdefinition does not maintain the selections of records from a position bigger than 76 
related to defect 0029028 acknowledged Triage Platform Base Selected record is lost after executing a pick and execute process under some circumnstances 
related to defect 0029145 closed Sandrahuguet Used credit is not correct under certain circumstances 
causes defect 0029032 closed AugustoMauch int-basic-pgsql/oracle tests are broken due to change in pick and execute windows 
causes defect 0029140 closed alostale Credit selected in the grid is not used when the payment is processed 
? com.openbravo.support.test-1.0.0.obx (23,955) 2015-01-21 12:38
https://issues.openbravo.com/file_download.php?file_id=7729&type=bug
Issue History
2015-01-21 12:38caristuNew Issue
2015-01-21 12:38caristuAssigned To => AugustoMauch
2015-01-21 12:38caristuFile Added: com.openbravo.support.test-1.0.0.obx
2015-01-21 12:38caristuModules => Core
2015-01-21 12:38caristuResolution time => 1421622000
2015-01-21 12:38caristuTriggers an Emergency Pack => No
2015-01-21 12:38caristuIssue Monitored: networkb
2015-01-21 12:38caristuRelationship addedrelated to 0028468
2015-01-21 12:48caristuResolution time1421622000 => 1425250800
2015-01-21 12:49heccamIssue Monitored: heccam
2015-01-29 12:26alostaleNote Added: 0073893
2015-01-29 12:27alostaleAssigned ToAugustoMauch => caristu
2015-01-29 12:27alostaleStatusnew => feedback
2015-01-29 13:07alostaleAssigned Tocaristu => alostale
2015-01-29 13:07alostaleStatusfeedback => new
2015-01-29 13:07alostaleStatusnew => acknowledged
2015-02-12 13:41egoitzRelationship addedrelated to 0028923
2015-02-17 13:40alostaleNote Deleted: 0073893
2015-02-19 12:52alostaleStatusacknowledged => scheduled
2015-02-19 12:52alostaleAssigned Toalostale => NaroaIriarte
2015-02-19 12:52alostalefix_in_branch => pi
2015-02-23 09:42alostaleRelationship addedrelated to 0022522
2015-02-23 10:19AugustoMauchAssigned ToNaroaIriarte => alostale
2015-02-23 11:11alostaleNote Added: 0074798
2015-02-23 11:11alostaleReview Assigned To => AugustoMauch
2015-02-23 11:11alostalefix_in_branchpi =>
2015-02-23 11:23hgbotCheckin
2015-02-23 11:23hgbotNote Added: 0074800
2015-02-23 11:23hgbotStatusscheduled => resolved
2015-02-23 11:23hgbotResolutionopen => fixed
2015-02-23 11:23hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d061c9e837aa5a0ce0a7fb221e1ac8c822748988 [^]
2015-02-23 15:46AugustoMauchNote Added: 0074811
2015-02-23 15:46AugustoMauchStatusresolved => new
2015-02-23 15:46AugustoMauchResolutionfixed => open
2015-02-23 16:17caristuRelationship addedrelated to 0029028
2015-02-23 16:18caristuRelationship deletedrelated to 0029028
2015-02-23 17:37alostaleRelationship addedrelated to 0029028
2015-02-23 17:48AugustoMauchStatusnew => scheduled
2015-02-23 17:48AugustoMauchfix_in_branch => pi
2015-02-23 17:49AugustoMauchNote Added: 0074813
2015-02-23 17:49AugustoMauchStatusscheduled => resolved
2015-02-23 17:49AugustoMauchResolutionopen => fixed
2015-02-23 18:13AugustoMauchRelationship addedcauses 0029032
2015-02-24 10:59AugustoMauchNote Added: 0074829
2015-02-24 10:59AugustoMauchStatusresolved => new
2015-02-24 10:59AugustoMauchResolutionfixed => open
2015-02-24 11:08hgbotCheckin
2015-02-24 11:08hgbotNote Added: 0074830
2015-02-24 11:28AugustoMauchNote Added: 0074838
2015-02-24 17:08alostaleStatusnew => scheduled
2015-02-24 23:09hudsonbotCheckin
2015-02-24 23:09hudsonbotNote Added: 0074889
2015-02-24 23:09hudsonbotCheckin
2015-02-24 23:09hudsonbotNote Added: 0074895
2015-02-27 12:43hgbotCheckin
2015-02-27 12:43hgbotNote Added: 0075028
2015-03-02 08:45hgbotCheckin
2015-03-02 08:45hgbotNote Added: 0075053
2015-03-02 08:45hgbotStatusscheduled => resolved
2015-03-02 08:45hgbotResolutionopen => fixed
2015-03-02 08:45hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/d061c9e837aa5a0ce0a7fb221e1ac8c822748988 [^] => http://code.openbravo.com/erp/devel/pi/rev/becbc7144120c7e6b95f93309b9d2617648547bf [^]
2015-03-02 13:04AugustoMauchNote Added: 0075075
2015-03-02 13:04AugustoMauchStatusresolved => closed
2015-03-02 13:04AugustoMauchFixed in Version => 3.0PR15Q2
2015-03-02 20:31hudsonbotCheckin
2015-03-02 20:31hudsonbotNote Added: 0075107
2015-03-05 12:34AugustoMauchRelationship addedcauses 0029140
2015-03-05 17:07alostaleRelationship addedrelated to 0029145

Notes
(0074798)
alostale   
2015-02-23 11:11   
Note that following the described steps till 5 and submitting the form, the record selected in a different page is not sent to the backend as part of the selection.
(0074800)
hgbot   
2015-02-23 11:23   
Repository: erp/devel/pi
Changeset: d061c9e837aa5a0ce0a7fb221e1ac8c822748988
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Mon Feb 23 11:17:39 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/d061c9e837aa5a0ce0a7fb221e1ac8c822748988 [^]

fixed bug 28712: record selection lost in p&e grid if data has multiple pages

  In P&E grid with data displayed in several pages, if part of the selection was
  not in the 1st page two errors occured:

    -If after selecting a record in 2nd page another one in the 1st page was selected,
     selection of record in 2nd page was lost
    -If record in 2nd page was selected by using a filter, the filter was removed, and
     the p&e was submitted, the selection was not sent to backend

  The problem is in SC grid.getSelectedRecords which returns records only in current
  page.

  To solve this a cache of selected/unselected records is completelly maintained for p&e
  grids without using getSelectedRecords, this is used also to send the selected recors
  on p&e submission.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-pickeditgrid.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-pick-and-execute-grid.js
---
(0074811)
AugustoMauch   
2015-02-23 15:46   
Reopened because apparently the fix has broken ci [1], [2]

[1] http://ci.openbravo.com/job/int-basic-pgsql/772/ [^]
[2] http://ci.openbravo.com/job/int-basic-oracle/780/ [^]
(0074813)
AugustoMauch   
2015-02-23 17:49   
Set as resolved again, the CI break will be dealt with in another issue
(0074829)
AugustoMauch   
2015-02-24 10:59   
Reopened because the fix has broken ci [1], [2], and fixing those broken tests is not trivial

[1] http://ci.openbravo.com/job/int-basic-pgsql/772/ [^]
[2] http://ci.openbravo.com/job/int-basic-oracle/780/ [^]
(0074830)
hgbot   
2015-02-24 11:08   
Repository: erp/devel/pi
Changeset: b8e08285349001bc1d72d2a0a49f40be59a1e2db
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Tue Feb 24 11:07:02 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b8e08285349001bc1d72d2a0a49f40be59a1e2db [^]

Fixes issue 29032, related with issue 28712: Backs out changeset d061c9e837aa

This changeset has been backed out as it is breaking the int-basic-pgsql and int-basic-oracle tests.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-pickeditgrid.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-pick-and-execute-grid.js
---
(0074838)
AugustoMauch   
2015-02-24 11:28   
This changeset [1] was causing at least one regression:

As QA Testing Admin:
- Open the Sales Order window
- Create a new record, use the Customer A business partner
- Create a new line
- Book the record
- Click on the Add Payment button. The first line of the P&E grid will be selected, but some events that should have been triggered have not taken place properly. The Expected Payment read-only parameter should be equals to the Amount column of the selected row, but it is empty.
- Deselect the row and select it again. Check that now the value of the Expected Payment parameter is OK.

[1] https://issues.openbravo.com/view.php?id=28712#c74800 [^]
(0074889)
hudsonbot   
2015-02-24 23:09   
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/6c8e140b31ec [^]
Maturity status: Test
(0074895)
hudsonbot   
2015-02-24 23:09   
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/6c8e140b31ec [^]
Maturity status: Test
(0075028)
hgbot   
2015-02-27 12:43   
Repository: erp/mods/com.openbravo.support.test
Changeset: 66afed4b598638dd0409fd4237e3e37b6b612c7b
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Fri Feb 27 11:53:49 2015 +0100
URL: http://code.openbravo.com/erp/mods/com.openbravo.support.test/rev/66afed4b598638dd0409fd4237e3e37b6b612c7b [^]

added p&e for issue 28712

---
R src-db/database/model/modifiedTables/C_ORDER.xml
R src-db/database/model/modifiedTables/M_INOUTLINE.xml
R src-db/database/model/views/SUPTEST_ORDERLINE_V.xml
R src-db/database/sourcedata/AD_COLUMN.xml
R src-db/database/sourcedata/AD_FIELD.xml
R src-db/database/sourcedata/AD_MENU.xml
R src-db/database/sourcedata/AD_MODULE.xml
R src-db/database/sourcedata/AD_MODULE_DBPREFIX.xml
R src-db/database/sourcedata/AD_MODULE_DEPENDENCY.xml
R src-db/database/sourcedata/AD_PACKAGE.xml
R src-db/database/sourcedata/AD_REFERENCE.xml
R src-db/database/sourcedata/AD_TAB.xml
R src-db/database/sourcedata/AD_TABLE.xml
R src-db/database/sourcedata/AD_TREENODE.xml
R src-db/database/sourcedata/AD_WINDOW.xml
R src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
R src-db/database/sourcedata/OBUIAPP_PROCESS.xml
R src-db/database/sourcedata/OBUIAPP_REF_WINDOW.xml
R src-db/database/sourcedata/OBUIAPP_VIEW_IMPL.xml
R src/TestButler.java
R src/TestDate.java
R src/com/openbravo/support/test/MyComponentProvider.java
R src/com/openbravo/support/test/SetStorageBinHandler.java
R src/com/openbravo/support/test/SupSelectOrderLines.java
R src/com/openbravo/support/test/SupSelectOrderLines.java~
R src/com/openbravo/support/test/TestButler.java
R web/com.openbravo.support.test/js/myCalendar.js
R web/com.openbravo.support.test/js/onSelect.js
R web/com.openbravo.support.test/js/onSelect.js~
---
(0075053)
hgbot   
2015-03-02 08:45   
Repository: erp/devel/pi
Changeset: becbc7144120c7e6b95f93309b9d2617648547bf
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Feb 26 11:26:38 2015 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/becbc7144120c7e6b95f93309b9d2617648547bf [^]

fixed bug 28712: record selection lost in p&e grid if data has multiple pages

  In P&E grid with data displayed in several pages, if part of the selection was
  not in the 1st page two errors occured:

    -If after selecting a record in 2nd page another one in the 1st page was selected,
     selection of record in 2nd page was lost
    -If record in 2nd page was selected by using a filter, the filter was removed, and
     the p&e was submitted, the selection was not sent to backend

  The problem is in SC grid.getSelectedRecords which returns records only in current
  page.

  To solve this a cache of selected/unselected records is completelly maintained for p&e
  grids without using getSelectedRecords, this is used also to send the selected recors
  on p&e submission.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-pickeditgrid.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-pick-and-execute-grid.js
---
(0075075)
AugustoMauch   
2015-03-02 13:04   
Code reviewed and verified in pi@fc4236ca4ff7
(0075107)
hudsonbot   
2015-03-02 20:31   
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/9d659cacae03 [^]
Maturity status: Test