Update of /var/cvs/contributions/natmm/src/nl/natuurmonumenten/activiteiten
In directory
james.mmbase.org:/tmp/cvs-serv26741/src/nl/natuurmonumenten/activiteiten
Modified Files:
Tag: BRANCH_NATMM_1_2
ActiviteitenHelper.java
Log Message:
NMCMS-622 - check op lege parameters toegevoegd, sommige clients geven
blijkbaar een lege string ("") als parameter ipv expliciet null
See also:
http://cvs.mmbase.org/viewcvs/contributions/natmm/src/nl/natuurmonumenten/activiteiten
See also: http://www.mmbase.org/jira/browse/NMCMS-622
Index: ActiviteitenHelper.java
===================================================================
RCS file:
/var/cvs/contributions/natmm/src/nl/natuurmonumenten/activiteiten/Attic/ActiviteitenHelper.java,v
retrieving revision 1.1.2.5
retrieving revision 1.1.2.6
diff -u -b -r1.1.2.5 -r1.1.2.6
--- ActiviteitenHelper.java 19 Jan 2009 13:55:47 -0000 1.1.2.5
+++ ActiviteitenHelper.java 30 Jan 2009 10:50:32 -0000 1.1.2.6
@@ -58,10 +58,14 @@
StringBuffer eventTypeConstraint = new StringBuffer();
if (eventTypeIds != null) {
- eventTypeConstraint.append("(");
boolean first = true;
for (int i = 0; i < eventTypeIds.length; i++) {
+ // sommige soap clients geven een array met 1 leeg element ipv
een lege array of null
+ if (isEmpty(eventTypeIds[i])) {
+ continue;
+ }
if (first) {
+ eventTypeConstraint.append("(");
first = false;
} else {
eventTypeConstraint.append(" OR ");
@@ -70,12 +74,14 @@
eventTypeConstraint.append(eventTypeIds[i]);
eventTypeConstraint.append("'");
}
+ if (!first) {
eventTypeConstraint.append(")");
}
+ }
logger.debug("eventTypeConstraint: " + eventTypeConstraint);
String provincieConstraint = null;
- if (provincieId != null) {
+ if (!isEmpty(provincieId)) {
provincieConstraint = "lokatie like '%," + provincieId + ",%'";
}
logger.debug("provincieConstraint: " + provincieConstraint);
@@ -101,7 +107,7 @@
}
logger.debug("natuurgebiedenId: " + natuurgebiedenId);
- if (natuurgebiedenId != null) {
+ if (!isEmpty(natuurgebiedenId)) {
parentBelongsToNatuurgebied = false;
NodeList list2 = cloud.getList(natuurgebiedenId,
"natuurgebieden,related,evenement", "evenement.number", "evenement.number='" +
parentNumber +"'", null, null, null, true);
if (!list2.isEmpty()) {
@@ -264,4 +270,9 @@
return thisParticipant;
}
+ private static boolean isEmpty(String str) {
+ return str == null || str.trim().length() == 0;
+ }
+
+
}
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs
