Commit Graph

5914 Commits

Author SHA1 Message Date
Redhuan D. Oon ede3856d8f JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11533&view=rev
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-28 02:03:00 +00:00
Redhuan D. Oon 583954e3eb FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test in next commit (no failures)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-28 02:01:59 +00:00
Carlos Ruiz 03153697fe Peer review 11500, minor - drop useless comment, organize imports
Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 21:06:50 +00:00
Carlos Ruiz cc7809b482 Apply Best Practices to Promotions Code: Use Model Classes instead of X_ classes whenever possible 2010-02-27 20:38:11 +00:00
Carlos Ruiz 1210dab275 Peer review 11504, be careful, you need to check retValue for null to avoid NPE
refactored to use common MProduct constructor
Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 19:54:34 +00:00
Carlos Ruiz 94812caebc Peer review 11501, be careful with the login being replaced
Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 19:38:25 +00:00
Carlos Ruiz 4a552b9877 Peer review 11499, fix error reported by Mario
Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 19:23:17 +00:00
mjudd baa578cdfb [ BF 2960239 ] - Promotion Minimum Amt > 0
In the promotion line if the minimum amount is 0.00 (default) then the line
is not fired because the Promotion rule only activates when the minimum
amount is > 0. This means if you always want the rule to fire - you need
to make the minimum 0.01 which is not logical if you want no minimum.
https://sourceforge.net/tracker/?func=detail&aid=2960239&group_id=176962&atid=879332
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2960239
2010-02-27 15:41:14 +00:00
mjudd 139f65a82e [ BF 2960231 ] - PromotionValidator not fired in correct client
The promotion validator document validator is not fired because the client
id in the listener is not set correctly before adding the table to the
Model Validation Engine
https://sourceforge.net/tracker/?func=detail&aid=2960231&group_id=176962&atid=879332
2010-02-27 15:21:37 +00:00
Redhuan D. Oon d940e4d3e9 JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11520&view=rev
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 15:05:00 +00:00
Redhuan D. Oon bb3a54da57 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test in next commit (no failures)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 14:05:19 +00:00
Redhuan D. Oon 9aa3c31c35 Prevent SQL Injection (advice from Victor Perez)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 09:02:32 +00:00
mjudd 38396298cd [ FR 2836965 ] Group1 and Group2 fields are not considered in Schema Lines (Fixed)
http://sourceforge.net/tracker/index.php?func=detail&aid=2836965&group_id=176962&atid=879335
[ FR 1871403 ] Adding new field of product into Price List Schema (Fixed)
http://sourceforge.net/tracker/index.php?func=detail&aid=1871403&group_id=176962&atid=879335
Converted to CPrepraedStatement to avoid string injection and fixed "AND" logic
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2836965
2010-02-27 08:10:40 +00:00
tspc cd4c15c059 Merge bug fixes in release not committed to trunk
merge -c 11478 -c 11471 -c 11374
BF2958108	Answer in Workflow Activities keeps adding Yes/NO
https://sourceforge.net/tracker/?func=detail&atid=955896&aid=2958108&group_id=176962

BF2957689	Material Receipt -> Create From -> Only same wh not working
https://sourceforge.net/tracker/?func=detail&atid=955896&aid=2957689&group_id=176962

BF2815307 Default Tax Error
https://sourceforge.net/tracker/?func=detail&atid=879332&aid=2815307&group_id=176962
2010-02-27 06:16:16 +00:00
vpj-cd ec4e2f798a FR: [ 2214883 ] Remove SQL code and Replace for Query
Applying defensive programming approach and ABP.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 06:12:53 +00:00
Redhuan D. Oon 998d5c20f5 Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Heng Sin's advice. No unit testing done but compiler tested OK.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 02:47:51 +00:00
Redhuan D. Oon d0fcc6068a JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11504&view=rev
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 02:29:28 +00:00
Redhuan D. Oon 0da2f340e7 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test in next commit (no failures)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-27 02:26:04 +00:00
Redhuan D. Oon 8df91cee04 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test included (no failures)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-26 11:39:31 +00:00
Redhuan D. Oon dd7b9b1341 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test included (no failures)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-26 10:21:04 +00:00
Redhuan D. Oon 5b537cf891 Typo error at MBOM 2010-02-26 07:27:43 +00:00
Redhuan D. Oon 7cd1ead853 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test included (no failures). Please clean up M_BOM test record afterwards.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-26 07:07:32 +00:00
Redhuan D. Oon 8cd9f28474 FR: [ 2214883 ] Remove SQL code and Replace for Query (implementing review by Carlos, Hengsin and Victor - call to interface rather than model class for tablename. JUnit tests carried out again before commit
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-26 02:51:35 +00:00
Heng Sin Low fa569115e1 The window context variable is share by all tab and can get mix up when user navigate from one tab to another. This patch recreate the window context variable before entering a tab to solve the potential conflict issue.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2959339
2010-02-26 02:07:55 +00:00
Heng Sin Low cd210e3170 minor - add undefined and null check 2010-02-26 01:52:35 +00:00
Heng Sin Low 1516f063b5 don't have to add setParameter(int), we just need to replace setParameters(Object[] parameters) with setParameters(Object ...parameters)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-26 01:02:06 +00:00
Heng Sin Low c7175c22ba Remove use of ejb, not necessary since zkwebui is already running at server.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2959285
2010-02-26 00:40:39 +00:00
Carlos Ruiz 0725d135a9 Peer review of FR: [ 2214883 ] Remove SQL code and Replace for Query
revisions 11482, 11484, 11485, 11487, 11488, 11489, 11490
- Added method Query.getParameters(int) to improve readability (if most of the replacements just use one int parameter it makes sense to avoid the declaration of new Object[] all the times)
- drop unused imports
- check serialVersionUID
- what is the advantage of using I_M_Product_BOM.Table_Name instead of MProductBOM.Table_Name when defining a list of MProductBOM objects? and it's intended that MProductBOM must always implement the interface

Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-26 00:29:57 +00:00
Heng Sin Low 47c6baa245 ValuePreference dialog missing the explaination message
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2959281
2010-02-26 00:24:49 +00:00
vpj-cd 1603524916 Apply ABP http://www.adempiere.com/index.php/ADempiere_Best_Practices#How_to_use_Adempiere_Query_class.3F
//FR: [ 2214883 ] Remove SQL code and Replace for Query

Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-25 21:32:06 +00:00
Redhuan D. Oon 398eaf6f10 FR: [ 2214883 ] Remove SQL code and Replace for Query (thanks Trifon for the last review and correction.
-- JUnit test included (no failures)
2010-02-25 21:08:04 +00:00
trifonnt 025e667dd2 FR: [ 2214883 ] Remove SQL code and Replace for Query
Please keep transaction.
Old code used transaction, so new should also use transaction.
http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-25 19:30:58 +00:00
trifonnt fea9618e40 FR: [ 2214883 ] Remove SQL code and Replace for Query
Please keep transaction.
old code used transaction, so new should also use transaction.
http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-25 19:27:08 +00:00
Redhuan D. Oon 011ec4a4d6 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test included (no failures)
2010-02-25 16:03:58 +00:00
Redhuan D. Oon 5d0fe54df9 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- JUnit test included (no failures)
2010-02-25 14:21:59 +00:00
mjudd d4834a9372 [ FR 2836965 ] Group1 and Group2 fields are not considered in Schema Lines (Fixed)
http://sourceforge.net/tracker/index.php?func=detail&aid=2836965&group_id=176962&atid=879335
[ FR 1871403 ] Adding new field of product into Price List Schema (Fixed)
http://sourceforge.net/tracker/index.php?func=detail&aid=1871403&group_id=176962&atid=879335
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2836965
2010-02-25 13:12:33 +00:00
Redhuan D. Oon 8aaa009751 FR: [ 2214883 ] Remove SQL code and Replace for Query
-- I am begining to use JUnit test and commit them as i go along also. Please advice where need be, or
-- If OK, i shall remove more SQLs from ADempiere :)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
2010-02-25 08:12:39 +00:00
Heng Sin Low 4b3d5de150 Dropdown sometime doesn't display the currently selected value ( swing
always does )
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2958556
2010-02-25 08:03:23 +00:00
Heng Sin Low 75f4dc9eba Info window pagign
- use cursor to load a subset of the result when native database paging is not available.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2913975
2010-02-24 08:18:47 +00:00
Heng Sin Low 2a74599b51 Fixed postgresql bug for virtual table:
* postgresql need trx to use cursor based resultset
* must set fetchsize for postgresql otherwise it will load the whole resultset into memory

Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2913975
2010-02-24 06:27:04 +00:00
Heng Sin Low da5292bbea turn on virtual table mode for swing in trunk for others to test
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2913975
2010-02-24 05:39:43 +00:00
phib eb40b62629 FR 2957782: Add due date to Pay selection create
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2957782
2010-02-24 05:22:02 +00:00
phib 4907114af1 BF 2957762: Payment selection print has the wrong cheque number
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2957762
2010-02-24 03:58:28 +00:00
phib c5f6c84434 BF 2957756: Payment selection print creates payments out of transaction
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2957756
2010-02-24 03:44:39 +00:00
trifonnt b6fe0e9ee8 BF [2957215] - Wrong #AD_User_ID when user with the same name from two Ten.
https://sourceforge.net/tracker/?func=detail&aid=2957215&group_id=176962&atid=879332
2010-02-23 13:34:19 +00:00
Heng Sin Low c7b42af6dd * Fixed version mixed up in previous commit.
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2956390
2010-02-23 13:20:41 +00:00
trifonnt f537c99a87 BF [2957094] - sql script fails on oracle
https://sourceforge.net/tracker/?func=detail&atid=879332&aid=2957094&group_id=176962
2010-02-23 09:21:11 +00:00
phib 24ee390c32 Fix SQL error with Merge entities and don't ignore some errors
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2936826
2010-02-23 01:55:33 +00:00
trifonnt b8f54de8f9 BF [2956546] - Login from WEB should also use LDAPUser
https://sourceforge.net/tracker/?func=detail&aid=2956546&group_id=176962&atid=879332
2010-02-22 12:14:39 +00:00
trifonnt bf14c33698 2010-02-22 12:01:31 +00:00