Permalink
Browse files

Log at error rather than warn when sites can't be found.

These sites are needed for the correct functioning of the hierarchy and
so things don't work well when they are missing.
  • Loading branch information...
1 parent eefbc65 commit de725cf610d5c6b6b22602baa78185196f4f49f8 @buckett buckett committed Mar 19, 2013
View
4 hierarchy-impl/impl/src/java/org/sakaiproject/hierarchy/impl/PortalHierarchyServiceImpl.java
@@ -150,15 +150,15 @@ private PortalNodeImpl populatePortalNodeSite(PortalPersistentNode portalPersist
Site missingSite = siteService.getSite(missingSiteId);
portalNode.setSite(missingSite);
} catch (IdUnusedException iue2 ) {
- log.warn("Couldn't find missing site "+ missingSiteId);
+ log.error("Couldn't find missing site "+ missingSiteId);
// Just return a node without a site.
}
}
try {
Site managementSite = siteService.getSite(portalPersistentNode.getManagementSiteId());
portalNode.setManagementSite(managementSite);
} catch (IdUnusedException e) {
- log.warn("Couldn't find management site "+ portalPersistentNode.getManagementSiteId()+ " for "+ portalPersistentNode.getPath());
+ log.error("Couldn't find management site "+ portalPersistentNode.getManagementSiteId()+ " for "+ portalPersistentNode.getPath());
}
return portalNode;
}

0 comments on commit de725cf

Please sign in to comment.