Except for it being touched occasionally by project-wide changes, I haven't seen any work done in the Shark component in years.

-Adrian

Jacques Le Roux wrote:
Oops, Sorry

I missed "> 0" when replacing xpdls != null && xpdls.size() > 0 by UtilValidate.isNotEmpty(xpdls)
Fixed at r884325

The reason of all these troubles is that Shark is not compiled anymore, not Workflow (this answer Adam question). Do we really need to keep them in trunk ? We may put them aside but still accessible ?

Jacques

From: "Scott Gray" <[email protected]>
Can a boolean be compared to an integer or did you just break it again?

Regards
Scott

HotWax Media
http://www.hotwaxmedia.com

On 26/11/2009, at 11:29 AM, [email protected] wrote:

Author: jleroux
Date: Wed Nov 25 22:29:21 2009
New Revision: 884308

URL: http://svn.apache.org/viewvc?rev=884308&view=rev
Log:
Complete Scott's fix in r884292 (using initial code in r821643

Modified:
ofbiz/trunk/specialpurpose/shark/src/org/ofbiz/shark/repository/ EntityRepositoryMgr.java

Modified: ofbiz/trunk/specialpurpose/shark/src/org/ofbiz/shark/ repository/EntityRepositoryMgr.java URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/shark/src/org/ofbiz/shark/repository/EntityRepositoryMgr.java?rev=884308&r1=884307&r2=884308&view=diff = = = = = = = = ====================================================================== --- ofbiz/trunk/specialpurpose/shark/src/org/ofbiz/shark/repository/ EntityRepositoryMgr.java (original) +++ ofbiz/trunk/specialpurpose/shark/src/org/ofbiz/shark/repository/ EntityRepositoryMgr.java Wed Nov 25 22:29:21 2009
@@ -177,7 +177,7 @@

public boolean doesXPDLExist(RepositoryTransaction t, String xpdlId) throws RepositoryException {
        List xpdls = this.getXpdlValues(xpdlId, null, false);
- Debug.log(UtilValidate.isNotEmpty(xpdls) ? true : false, module); + Debug.log("Does XPDL [" + xpdlId + "] Exist - " + xpdls + "(" + (UtilValidate.isNotEmpty(xpdls) > 0 ? true : false) + ")", module);
        return (UtilValidate.isNotEmpty(xpdls) ? true : false);
    }








Reply via email to