Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013472Openbravo ERP00. Application dictionarypublic2010-05-28 18:462010-07-15 00:00
Reporternetworkb 
Assigned Toalostale 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.50MP18 
Target Version2.50MP20Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0013472: You can delete records of a table with 'Deletable Records' unchecked
DescriptionYou can delete records of a table with 'Deletable Records' unchecked
Steps To Reproduce- Create a new table on your DB
- Create the table on AD with 'Deletable Records' unchecked.
- Create a window and a Menu
- Compile
- Restart Tomcat
- Go to the table created insert some records and then try to delete them.
- Realize that this operation is possible
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-05-28 18:46networkbNew Issue
2010-05-28 18:46networkbAssigned To => alostale
2010-05-28 18:46networkbOBNetwork customer => Yes
2010-05-31 08:31alostaleStatusnew => scheduled
2010-05-31 08:31alostalefix_in_branch => pi
2010-07-06 16:50hgbotCheckin
2010-07-06 16:50hgbotNote Added: 0029096
2010-07-06 16:50hgbotStatusscheduled => resolved
2010-07-06 16:50hgbotResolutionopen => fixed
2010-07-06 16:50hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/6e5b61d7248a52057d0d4e8436a45939e43c3594 [^]
2010-07-07 12:38hudsonbotCheckin
2010-07-07 12:38hudsonbotNote Added: 0029122
2010-07-14 11:37marvintmNote Added: 0029305
2010-07-14 11:37marvintmStatusresolved => closed
2010-07-15 00:00anonymoussf_bug_id0 => 3029717

Notes
(0029096)
hgbot   
2010-07-06 16:50   
Repository: erp/devel/pi
Changeset: 6e5b61d7248a52057d0d4e8436a45939e43c3594
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Jul 06 16:49:45 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/6e5b61d7248a52057d0d4e8436a45939e43c3594 [^]

fixed issue 13472: Don't allow delete in tabs without deleteable records

  Tabs genereated for tables without deleteable records field checked, don't
  allow now record deletion:
    -Delete icon has been removed from toolbar for these cases.
    -Logic has been implemented to preven deletion from grid and edit views.

---
M src-wad/src/org/openbravo/wad/Tabs_data.xsql
M src-wad/src/org/openbravo/wad/Wad.java
M src-wad/src/org/openbravo/wad/javasource.javaxml
M src-wad/src/org/openbravo/wad/javasource.xml
M src/org/openbravo/erpCommon/utility/DataGrid.java
M src/org/openbravo/erpCommon/utility/ToolBar.java
M src/org/openbravo/erpCommon/utility/WindowAccess_data.xsql
---
(0029122)
hudsonbot   
2010-07-07 12:38   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/6e5b61d7248a [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/d555a35070ee [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17817.obx [^]
(0029305)
marvintm   
2010-07-14 11:37   
Verified that indeed when you change this property and you compile the window, the Delete button doesn't show anymore. Even more, if you don't compile the window, but try to delete a row after you've changed the Deletable Rows property, you still get an error message, so it's working very well now.