midsuit
/
core-jgi
6
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

added null check for Parent_ID element.

This commit is contained in:
Heng Sin Low 2010-12-24 11:55:02 +08:00
parent 7d4f9afb08
commit cc04468c7d
1 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
org.adempiere.pipo.handlers/src/org/adempiere/pipo2/handler/MenuElementHandler.java
View File

@ -54,7 +54,7 @@ public class MenuElementHandler extends AbstractElementHandler {
public void startElement(Properties ctx, Element element) public void startElement(Properties ctx, Element element)
throws SAXException { throws SAXException {
List<String> excludes = defaultExcludeList(X_AD_Menu.Table_Name); List<String> excludes = defaultExcludeList(X_AD_Menu.Table_Name);
X_AD_Menu mMenu = findPO(ctx, element); X_AD_Menu mMenu = findPO(ctx, element);
@ -69,7 +69,7 @@ public class MenuElementHandler extends AbstractElementHandler {
{ {
filler.setInteger("AD_Menu_ID"); filler.setInteger("AD_Menu_ID");
} }
List<String> notFounds = filler.autoFill(excludes); List<String> notFounds = filler.autoFill(excludes);
if (notFounds.size() > 0) { if (notFounds.size() > 0) {
element.defer = true; element.defer = true;
@ -78,7 +78,7 @@ public class MenuElementHandler extends AbstractElementHandler {
if (!mMenu.is_new() && !mMenu.is_Changed()) if (!mMenu.is_new() && !mMenu.is_Changed())
return; return;
X_AD_Package_Imp_Detail impDetail = createImportDetail(ctx, element.qName, X_AD_Menu.Table_Name, X_AD_Package_Imp_Detail impDetail = createImportDetail(ctx, element.qName, X_AD_Menu.Table_Name,
X_AD_Menu.Table_ID); X_AD_Menu.Table_ID);
String action = null; String action = null;
@ -103,14 +103,16 @@ public class MenuElementHandler extends AbstractElementHandler {
log.info("setmenu:" + e); log.info("setmenu:" + e);
} }
} }
Element parentElement = element.properties.get("Parent_ID"); Element parentElement = element.properties.get("Parent_ID");
int parentId = 0; int parentId = 0;
if (ReferenceUtils.isIDLookup(parentElement) || ReferenceUtils.isUUIDLookup(parentElement)) { if (parentElement != null) {
parentId = ReferenceUtils.resolveReference(ctx, parentElement); if (ReferenceUtils.isIDLookup(parentElement) || ReferenceUtils.isUUIDLookup(parentElement)) {
} else { parentId = ReferenceUtils.resolveReference(ctx, parentElement);
String parent = getStringValue(element, "Parent_ID"); } else {
parentId = findIdByName(ctx, "AD_Menu", parent); String parent = getStringValue(element, "Parent_ID");
parentId = findIdByName(ctx, "AD_Menu", parent);
}
} }
StringBuffer updateSQL = null; StringBuffer updateSQL = null;