Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0005541Openbravo ERPA. Platformpublic2008-10-15 17:102008-11-26 20:20
Reportershuehner 
Assigned Toshuehner 
PrioritynoneSeveritytrivialReproducibilityN/A
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version2.50Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0005541: Cleanup: Update users of SequenceIdData.getSequence/getSequenceConnection to directly use getUUID
DescriptionThe function getSequence is deprecated and just calls getUUID after the uuid merge. The function getSequenceConnection calls the database function AD_SEQUENCE_NEXT with just call the database function get_uuid (after the uuid merge). This last one did this within the scope of the specified transaction, which does not matter anymore with the randomly allocated UUID's.

All java callers should be updated to directly call getUUID().
Steps To Reproduce
Proposed Solution
Additional Information
TagsUUID project
Relationships
related to feature request 0004795 acknowledged Triage Platform Base Remove deprecated ad_get_next_sequence calls and use get_uuid instead (in r2.5x) 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-10-15 17:10shuehnerNew Issue
2008-10-15 17:10shuehnerAssigned To => shuehner
2008-10-15 17:10shuehnersf_bug_id0 => 2168709
2008-10-15 17:10shuehnerRegression testing => No
2008-10-15 17:10shuehnerTag Attached: UUID project
2008-10-15 17:13svnbotCheckin
2008-10-15 17:13svnbotNote Added: 0009588
2008-10-15 17:13svnbotsvn_revision => 8789
2008-10-15 17:16shuehnerStatusnew => scheduled
2008-10-15 17:16shuehnerfix_in_branch => trunk
2008-10-15 17:19svnbotCheckin
2008-10-15 17:19svnbotNote Added: 0009589
2008-10-15 17:19svnbotsvn_revision8789 => 8790
2008-10-15 17:47svnbotCheckin
2008-10-15 17:47svnbotNote Added: 0009590
2008-10-15 17:47svnbotStatusscheduled => resolved
2008-10-15 17:47svnbotResolutionopen => fixed
2008-10-15 17:47svnbotsvn_revision8790 => 8792
2008-10-15 17:56shuehnerRelationship addedrelated to 0004795
2008-11-26 20:20pjuvaraStatusresolved => closed

Notes
(0009588)
svnbot   
2008-10-15 17:13   
Repository: openbravo
Revision: 8789
Author: huehner
Date: 2008-10-15 17:13:41 +0200 (Wed, 15 Oct 2008)

Issue 5541
- Change callers of getSequence to directly call getUUID()
- mark getSequenec with @deprecated

---
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromInvoice.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromOrder.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromPOOrder.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CopyFromSettlement.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateCloseFactAcct.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateFrom.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateFromMultiple.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateRegFactAcct.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/CreateVatRegisters.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/ExpenseSOrder.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/Posted.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/ProjectCopyFrom.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/ProjectSetType.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/RequestAction.java
U trunk/src/org/openbravo/erpCommon/ad_actionButton/Test_In.java
U trunk/src/org/openbravo/erpCommon/ad_background/PeriodicAlert.java
U trunk/src/org/openbravo/erpCommon/ad_background/PeriodicBackground.java
U trunk/src/org/openbravo/erpCommon/ad_forms/AcctServer.java
U trunk/src/org/openbravo/erpCommon/ad_forms/CallAcctServer.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocAmortization.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocBank.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocCash.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocDPManagement.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocGLJournal.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocInOut.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocInventory.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocInvoice.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocMovement.java
U trunk/src/org/openbravo/erpCommon/ad_forms/DocPayment.java
U trunk/src/org/openbravo/erpCommon/ad_forms/EdiFileImport.java
U trunk/src/org/openbravo/erpCommon/ad_forms/FactLine.java
U trunk/src/org/openbravo/erpCommon/ad_forms/FileImport.java
U trunk/src/org/openbravo/erpCommon/ad_forms/GenerateInvoicesmanual.java
U trunk/src/org/openbravo/erpCommon/ad_forms/GenerateShipmentsmanual.java
U trunk/src/org/openbravo/erpCommon/ad_forms/InitialClientSetup.java
U trunk/src/org/openbravo/erpCommon/ad_forms/InvoiceVendorMultiline.java
U trunk/src/org/openbravo/erpCommon/ad_forms/InvoiceVendorMultiline_Lines.java
U trunk/src/org/openbravo/erpCommon/ad_forms/MaterialReceiptPending.java
U trunk/src/org/openbravo/erpCommon/ad_forms/RemittanceCancel.java
U trunk/src/org/openbravo/erpCommon/ad_forms/RequisitionToOrder.java
U trunk/src/org/openbravo/erpCommon/ad_forms/ShowSessionPreferences.java
U trunk/src/org/openbravo/erpCommon/ad_process/CashBankOperations.java
U trunk/src/org/openbravo/erpCommon/ad_process/ChangeOrderOrg.java
U trunk/src/org/openbravo/erpCommon/ad_process/CopyFromGLJournal.java
U trunk/src/org/openbravo/erpCommon/ad_process/CreateCashFlowStatement.java
U trunk/src/org/openbravo/erpCommon/ad_process/DeleteClient.java
U trunk/src/org/openbravo/erpCommon/ad_process/ExpenseAPInvoice.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportAccount.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportBPartner.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportBudget.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportOrder.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportProcess.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportProduct.java
U trunk/src/org/openbravo/erpCommon/ad_process/ImportTax.java
U trunk/src/org/openbravo/erpCommon/ad_process/PriceListCreateAll.java
U trunk/src/org/openbravo/erpCommon/ad_process/RequestReOpen.java
U trunk/src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.java
U trunk/src/org/openbravo/erpCommon/businessUtility/TabAttachments.java
U trunk/src/org/openbravo/erpCommon/info/AttributeSetInstance.java
U trunk/src/org/openbravo/erpCommon/info/Location.java
U trunk/src/org/openbravo/erpCommon/security/SessionLogin.java
U trunk/src/org/openbravo/erpCommon/utility/ExecuteTask.java
U trunk/src/org/openbravo/erpCommon/utility/SequenceIdData.java
U trunk/src/org/openbravo/erpCommon/ws/externalSales/ExternalSalesImpl.java
U trunk/src-wad/src/org/openbravo/wad/ActionButton_Responser.javaxml
---

https://dev.openbravo.com/websvn/openbravo/?rev=8789&sc=1 [^]
(0009589)
svnbot   
2008-10-15 17:19   
Repository: openbravo
Revision: 8790
Author: huehner
Date: 2008-10-15 17:19:17 +0200 (Wed, 15 Oct 2008)

Issue 5541
Change the only caller of getSequenceConnection to directly call getUUID()

---
U trunk/src/org/openbravo/erpCommon/ad_forms/RemittanceCancel.java
---

https://dev.openbravo.com/websvn/openbravo/?rev=8790&sc=1 [^]
(0009590)
svnbot   
2008-10-15 17:47   
Repository: openbravo
Revision: 8792
Author: huehner
Date: 2008-10-15 17:47:46 +0200 (Wed, 15 Oct 2008)

Fixed 5541
- mark getSequenceConnection as deprecated and let it call directly the java getUUID
  (instead of db-call ad_sequence_next db-call get_uuid()
- remove leftovers from sqlc-generation
  - remove dummy field and getter
  - remove implementes FieldProvider
- remove the Sequence_data.xsql file completely as the last and only caller
  was the code in SequenceIdData (which is no gone, too)

---
U trunk/src/org/openbravo/erpCommon/utility/SequenceIdData.java
D trunk/src/org/openbravo/erpCommon/utility/Sequence_data.xsql
---

https://dev.openbravo.com/websvn/openbravo/?rev=8792&sc=1 [^]