details:   https://code.openbravo.com/erp/devel/pi/rev/562813057ec3
changeset: 17626:562813057ec3
user:      Augusto Mauch <augusto.mauch <at> openbravo.com>
date:      Wed Aug 08 08:57:23 2012 +0200
summary:   Fixes issue 21306: Delete record shortcut improved

So far, the only check done when deleting a record with the shortcut Supr was 
to count the number of selected records, and not to check if the tab is read 
only, is the table is deleteable, etc.

Now, before deleting a record with the shortcut, the status of the delete 
button (where the logic to see if the removal of records is anabled) is 
checked. If the delete button is not enabled, it will not be possible to delete 
the record with the shortcut.

diffstat:

 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
 |  3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diffs (13 lines):

diff -r 6ec538cab068 -r 562813057ec3 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
--- 
a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
     Wed Aug 08 08:47:14 2012 +0200
+++ 
b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
     Wed Aug 08 08:57:23 2012 +0200
@@ -754,7 +754,8 @@
     });
 
     ksAction_DeleteSelectedRecords = function () {
-      if (me.getSelectedRecords().length > 0) {
+      var isDeletingEnabled = 
!me.view.toolBar.getLeftMember(isc.OBToolbar.TYPE_DELETE).disabled;
+      if (me.getSelectedRecords().length > 0 && isDeletingEnabled) {
         me.view.deleteSelectedRows();
         return false; //To avoid keyboard shortcut propagation
       } else {

------------------------------------------------------------------------------
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