Thu, 23 Jun 2016 18:31:34 +0200fixes issue 33343: read only logic in P&E param based on aux input doesn't work
Carlos Aristu <carlos.aristu@openbravo.com> [Thu, 23 Jun 2016 18:31:34 +0200] rev 29765
fixes issue 33343: read only logic in P&E param based on aux input doesn't work

Fixed by getting context info with setting properties param.

Fri, 08 Jul 2016 13:42:14 +0200related to bug 33200: getId in DAL proxy loads object in memory
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Jul 2016 13:42:14 +0200] rev 29764
related to bug 33200: getId in DAL proxy loads object in memory

Do not use DalUtil.getId: removed unneded DalUtil imports no longer required

Fri, 08 Jul 2016 13:41:24 +0200related to bug 33200: getId in DAL proxy loads object in memory
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Jul 2016 13:41:24 +0200] rev 29763
related to bug 33200: getId in DAL proxy loads object in memory

Do not use DalUtil.getId: fixed the rest of the cases the regexp couldn't
find.

Fri, 08 Jul 2016 13:40:39 +0200related to bug 33200: getId in DAL proxy loads object in memory
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Jul 2016 13:40:39 +0200] rev 29762
related to bug 33200: getId in DAL proxy loads object in memory

Do not use DalUtil.getId: massive update with regexp

Fri, 08 Jul 2016 13:40:12 +0200related to bug 33200: getId in DAL proxy loads object in memory
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Jul 2016 13:40:12 +0200] rev 29761
related to bug 33200: getId in DAL proxy loads object in memory

Do not use DalUtil.getId: fixed some special cases

Fri, 03 Jun 2016 15:16:08 +0200fixed bug 33200: getId in DAL proxy loads object in memory
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 03 Jun 2016 15:16:08 +0200] rev 29760
fixed bug 33200: getId in DAL proxy loads object in memory

Whenever getId method was invoked on a DAL proxy, the full object was initialized
in memory from DB. This is not necessary as the id information is part of the
proxy itself. This behavior could be workarounded by using DalUtil.getId method.

Now it has been fixed so that proxy.getId() does not load the object from DB,
because of this DalUtil.getId method is no longer needed, thus it has been
deprecated.

The fix consists in letting Hibernate know which is the getId method so it does
not require to load it.

Mon, 11 Jul 2016 08:33:18 +0200Merge back from main
RM packaging bot <staff.rm@openbravo.com> [Mon, 11 Jul 2016 08:33:18 +0200] rev 29759
Merge back from main

Mon, 11 Jul 2016 06:24:38 +0000Merge temporary head for 3.0PR16Q2.1
RM packaging bot <staff.rm@openbravo.com> [Mon, 11 Jul 2016 06:24:38 +0000] rev 29758
Merge temporary head for 3.0PR16Q2.1

Thu, 07 Jul 2016 05:58:30 +0000Added signature for changeset ac4b84085793
RM packaging bot <staff.rm@openbravo.com> [Thu, 07 Jul 2016 05:58:30 +0000] rev 29757
Added signature for changeset ac4b84085793

Thu, 07 Jul 2016 05:58:30 +0000Added tag 3.0PR16Q2.1 for changeset 7667e8781815
RM packaging bot <staff.rm@openbravo.com> [Thu, 07 Jul 2016 05:58:30 +0000] rev 29756
Added tag 3.0PR16Q2.1 for changeset 7667e8781815

Thu, 07 Jul 2016 05:58:29 +0000Update AD_MODULE version to 3.0PR16Q2.1 3.0PR16Q2.1
RM packaging bot <staff.rm@openbravo.com> [Thu, 07 Jul 2016 05:58:29 +0000] rev 29755
Update AD_MODULE version to 3.0PR16Q2.1

Thu, 16 Jun 2016 10:04:13 +0200fixed issue 33434: SqlC does execute all queries at compile time
Stefan Hühner <stefan.huehner@openbravo.com> [Thu, 16 Jun 2016 10:04:13 +0200] rev 29754
fixed issue 33434: SqlC does execute all queries at compile time

SqlC does execute all select queries at compile time when it is processing
xsql files.

Technically that is needed to get the number, names & data-types of the columns
in the result as that info is used to generate the fields and data-reading code
in the to be generated java file.

However the jdbc spec does have a optional method to get that same information
for a query without executing it.

As this method is supported in current jdbc drivers for both Oracle and PostgreSQL,
now it is used instead of executing the whole queries.

Tue, 05 Jul 2016 12:48:48 +0200Fixed issue 33420: Sometimes synchronization of the syncmodels are failing
Jorge Garcia <jorge.garcia@openbravo.com> [Tue, 05 Jul 2016 12:48:48 +0200] rev 29753
Fixed issue 33420: Sometimes synchronization of the syncmodels are failing
when returning back from offline

Added a new column in c_import_entry table wich contains the timestamp in
miliseconds of the import entry.

With this new value, the import entry records are filtered.

Thu, 30 Jun 2016 18:04:33 +0200Fixes issue 33387: Wrong Storage Bin in Goods Receipt Lines
Sanjota <sanjota.nelagi@promantia.com> [Thu, 30 Jun 2016 18:04:33 +0200] rev 29752
Fixes issue 33387: Wrong Storage Bin in Goods Receipt Lines

In some cases, when you create a goods receipt line the storage bin's defaul value don't belongs to the warehouse defined in the header.

Thu, 30 Jun 2016 13:17:45 +0200Related to issue 33380: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Thu, 30 Jun 2016 13:17:45 +0200] rev 29751
Related to issue 33380: Code review improvements

Thu, 30 Jun 2016 12:35:54 +0200Fixes issue 33380: Error! tag shown instead of Organization
Sanjota <sanjota.nelagi@promantia.com> [Thu, 30 Jun 2016 12:35:54 +0200] rev 29750
Fixes issue 33380: Error! tag shown instead of Organization

Error! tag shown instead of Organization in Initial Organization Setup success message.

Tue, 28 Jun 2016 10:05:28 +0200Related to issue 33361: Update copyright
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Tue, 28 Jun 2016 10:05:28 +0200] rev 29749
Related to issue 33361: Update copyright

Mon, 27 Jun 2016 11:57:06 +0200Fixed issue 33361: Cash VAT accounting with several ledgers
Víctor Martínez Romanos <victor.martinez@openbravo.com> [Mon, 27 Jun 2016 11:57:06 +0200] rev 29748
Fixed issue 33361: Cash VAT accounting with several ledgers

Comparing objects was wrongly done using equals instead of comparing by the ID.
The iteration for the first accounting schema worked fine, but the next ones were identified as other objects (although the record ID was the same).
Now the comparation is done using the ID.

Fri, 24 Jun 2016 09:39:53 -0400Fixes issue 33349: Allow to Void a NOT PAID invoice, logged as * organization
Mark <markmm82@gmail.com> [Fri, 24 Jun 2016 09:39:53 -0400] rev 29747
Fixes issue 33349: Allow to Void a NOT PAID invoice, logged as * organization

Was avoid the Error when trying to Void a NOT PAID invoice, logged with * organization. The payment detail takes the organization from the payment schedule details organization.

Wed, 22 Jun 2016 18:18:48 +0200Related to issue 33309: Code review improvements
Alvaro Ferraz <alvaro.ferraz@openbravo.com> [Wed, 22 Jun 2016 18:18:48 +0200] rev 29746
Related to issue 33309: Code review improvements

Use FIN_Payment_ScheduleDetail instead of e to avoid errors when compiling in 2.50 mode.

Mon, 20 Jun 2016 11:25:12 -0400Fixes issue 33309: Linked items corrected in Payment Plan when unpaid invoice
Mark <markmm82@gmail.com> [Mon, 20 Jun 2016 11:25:12 -0400] rev 29745
Fixes issue 33309: Linked items corrected in Payment Plan when unpaid invoice

Defined SQL Where Clause for Lines tab in Payment In and Payment Out window due that that HQL Where clause defined for Lines tab in Payment window is not executed from Linked Items correctly.

Fri, 17 Jun 2016 08:52:10 +0200fixes issue 33208: Direct link to documents in the workspace is not working
Carlos Aristu <carlos.aristu@openbravo.com> [Fri, 17 Jun 2016 08:52:10 +0200] rev 29744
fixes issue 33208: Direct link to documents in the workspace is not working

The problem happened having a saved view with a date field as the first one of the grid. In this case the date item was firing an extra datasource call causing the bad behavior. This extra call was being performed because the date field was detecting wrongly the deletion of its content. Now an extra check is added: the empty filter is applied if the newValue is '' and also if the grid was being filtered by this field.

Fri, 17 Jun 2016 08:39:55 +0200fixes issue 33225: ConcatPDFReport can throw a null pointer exception
Carlos Aristu <carlos.aristu@openbravo.com> [Fri, 17 Jun 2016 08:39:55 +0200] rev 29743
fixes issue 33225: ConcatPDFReport can throw a null pointer exception
Use the correct variable name (configuration instead of reportConfiguration) when using the setCreatingBatchModeBookmarks method

Thu, 07 Jul 2016 18:32:05 +0530Fixes Issue 32560:Allow to invoice an already invoiced goods receipt
Atul Gaware <atul.gaware@openbravo.com> [Thu, 07 Jul 2016 18:32:05 +0530] rev 29742
Fixes Issue 32560:Allow to invoice an already invoiced goods receipt

when creating goods receipt using "Pending Goods Receipt", associated
purchase invoice line with related purchase order line is updated with
the goods receipt line reference.

Fri, 08 Jul 2016 08:52:33 +0200related to bug 33003: small code style changes
Asier Lostalé <asier.lostale@openbravo.com> [Fri, 08 Jul 2016 08:52:33 +0200] rev 29741
related to bug 33003: small code style changes

- Renamed formatedPropKey to formattedPropKey
- formattedPropKey does not need to be initialized with "" if it is always
assigned with the correct value

Thu, 07 Jul 2016 17:56:26 +0200Fixes issue 33448: DatasetServiceTest checks only modules in OB3 distribution
Augusto Mauch <augusto.mauch@openbravo.com> [Thu, 07 Jul 2016 17:56:26 +0200] rev 29740
Fixes issue 33448: DatasetServiceTest checks only modules in OB3 distribution

The DatasetServiceTest class takes for granted that some rules applies to all datasets (i.e. that the dataset table where clause can be executed as is), but that cannot be met in some modules outside the OB3 distribution.

Now the test only checks the datasets defined in the modules included in the OB3 distribution.

Thu, 07 Jul 2016 17:50:11 +0200Related with issue 33448: Removes commented out code
Augusto Mauch <augusto.mauch@openbravo.com> [Thu, 07 Jul 2016 17:50:11 +0200] rev 29739
Related with issue 33448: Removes commented out code

Thu, 07 Jul 2016 17:16:11 +0200Fixes issue 33447:The List of modules included in OB3 distribution is available
Augusto Mauch <augusto.mauch@openbravo.com> [Thu, 07 Jul 2016 17:16:11 +0200] rev 29738
Fixes issue 33447:The List of modules included in OB3 distribution is available

The list of modules included in the OB3 distribution are now available using the ModulesInOB3Distribution.getModules() method. The list has been extracted from the HelpWindow cla
ss, which now references the extracted listFixes issue 33447:The List of modules included in OB3 distribution is available

The list of modules included in the OB3 distribution are now available using the ModulesInOB3Distribution.getModules() method. The list has been extracted from the HelpWindow cla
ss, which now references the extracted list.

Wed, 06 Jul 2016 17:52:48 +0200Fixed issue 32260: Editing records in grid view was not working properly.
Naroa Iriarte <naroa.iriarte@openbravo.com> [Wed, 06 Jul 2016 17:52:48 +0200] rev 29737
Fixed issue 32260: Editing records in grid view was not working properly.

Editing in grid view was not working properly when all the fields of the record were readonly.
The problem was in "ob-view-grid.js" in the "startEditing" function. In this function, the focused field is set, but when every field is readonly there was not setting the value, so it was undefined. This was causing the failure.

To fix this a new condition has been added. Now, it is checked if the colNum is undefined and if the new function "isThereAnyEditableField" returns false. If these conditions are met, it means that the focus is not set yet and that there aren't any editable fields. Because if there were any editable fields, not setting the focus at that point would not be a problem, smartclient puts the focus there.

Wed, 06 Jul 2016 16:26:46 +0200Related to issue 33273: code review improvements
Víctor Martínez Romanos <victor.martinez@openbravo.com> [Wed, 06 Jul 2016 16:26:46 +0200] rev 29736
Related to issue 33273: code review improvements

Avoid to run the isReversedInvoice() method several times