midsuit
/
core-jgi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
5,790 Commits 1 Branch 0 Tags 397 MiB
Commit Graph

5790 Commits

Author SHA1 Message Date
Carlos Ruiz ff9dffd082 peer review for 11541 - be careful with logic: an exception is not equivalent to a log.info 
refactored to use .setClientID()
FR: [ 2214883 ] Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-03 22:13:54 +00:00
Carlos Ruiz e7177049db peer review for 11533, 11537, 11540 
FR: [ 2214883 ] Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-03 21:43:37 +00:00
Carlos Ruiz d9b22ee7ed peer review for 11539 
FR: [ 2214883 ] Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-03 21:13:05 +00:00
Carlos Ruiz 791f4a8624 peer review for 11538 -> drop unused lines in ExportHelper - apply the same practice to ImportHelper, fix warning on ExportFormatGenerator 
FR: [ 2214883 ] Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-03 20:09:00 +00:00
Carlos Ruiz 6b59db69ed peer review for 11535 -> getBPartnerCashTrx refactored with junit test applied 
FR: [ 2214883 ] Remove SQL code and Replace for Query
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-03 19:47:16 +00:00
Redhuan D. Oon 63780c78b2 JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11559&view=rev 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-03 07:53:05 +00:00
Redhuan D. Oon ab5b31fe9a 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-03-03 07:51:30 +00:00
Carlos Ruiz dfe03fabfa FR 2962094 - Finish implementation of weighted average costing 
1st step -> create column to manage the processed date+time (in order to allow reposting of documents in the same order they were completed - this will be the second step)
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2962094
 2010-03-03 01:31:36 +00:00
phib 5fdefd4be3 Future dated price list version used in order 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2962244
 2010-03-02 22:59:41 +00:00
vpj-cd d6e3780968 Apply ADempiere Best Practice and change the Collection by List as standard in ADempiere 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-02 17:15:59 +00:00
Redhuan D. Oon 3486415202 Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Best Practice. No unit testing done but compiler tested OK. 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-02 15:44:42 +00:00
Redhuan D. Oon 7fb5e6b54d Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Best Practice. No unit testing done but compiler tested OK. 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-02 15:43:25 +00:00
Redhuan D. Oon d2333e9026 Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Best Practice. No unit testing done but compiler tested OK. 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-02 15:38:35 +00:00
Carlos Ruiz 4d76cea47e Found the NOT EXISTS part of the validation is not needed as it's controlled in MRMALine.beforeSave - better drop the NOT EXISTS part for backporting compatibility with 343 and 355 
Fix [2904257] wrong validation sql for "M_InOutShipment/Receipt (RMA)"
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2904257
 2010-03-01 17:30:14 +00:00
Redhuan D. Oon 5fba554eab Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Best Practice. No unit testing done but compiler tested OK. 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-01 07:18:20 +00:00
Redhuan D. Oon ce29e05125 JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11545&view=rev 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-01 06:42:06 +00:00
Redhuan D. Oon 83154f494f 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-03-01 06:39:45 +00:00
Heng Sin Low c3805382ae Prepare window context before entering a tab 
- Fixed invalid context for tab 0
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2959339
 2010-03-01 05:37:59 +00:00
Redhuan D. Oon bff8e97dd6 JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11541&view=rev 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-03-01 01:41:37 +00:00
Redhuan D. Oon c0d989b240 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-03-01 00:42:03 +00:00
Redhuan D. Oon a3cf7e8d37 Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Best Practice. No unit testing done but compiler tested OK. 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-02-28 22:41:26 +00:00
Redhuan D. Oon b690a934d3 Review FR: [ 2214883 ] Remove SQL code and Replace for Query >> to incorporate Best Practice. No unit testing done but compiler tested OK. 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-02-28 09:55:38 +00:00
Redhuan D. Oon 042083872b 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-28 05:12:58 +00:00
Redhuan D. Oon 490cb806df 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-28 04:39:28 +00:00
Redhuan D. Oon fdfe4fb8b1 JUNIT TEST for http://adempiere.svn.sourceforge.net/adempiere/?rev=11535&view=rev 
Link to SF Tracker: http://sourceforge.net/support/tracker.php?aid=2214883
 2010-02-28 04:26:18 +00:00
Redhuan D. Oon e80c558107 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 04:24:19 +00:00
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