danwatford commented on a change in pull request #252: URL: https://github.com/apache/ofbiz-framework/pull/252#discussion_r560122119
########## File path: applications/order/groovyScripts/order/CompanyHeader.groovy ########## @@ -135,12 +136,22 @@ if (!partyId) { } // the logo -partyGroup = from("PartyGroup").where("partyId", partyId).queryOne() +GenericValue partyGroup = from("PartyGroup").where("partyId", partyId).queryOne() if (partyGroup) { - partyContentWrapper = new PartyContentWrapper(dispatcher, partyGroup, locale, EntityUtilProperties.getPropertyValue("content", "defaultMimeType", "text/html; charset=utf-8", delegator)) - partyContent = partyContentWrapper.getFirstPartyContentByType(partyGroup.partyId , partyGroup, "LGOIMGURL", delegator) + GenericValue partyContent = PartyContentWrapper.getFirstPartyContentByType(partyId, partyGroup, "LGOIMGURL", delegator) Review comment: I can take another look, but before I do so what sort of URL would you expect from getPartyContentAsText? If the URL is of the form https://localhost/content/control/stream?contentId=10001 then this won't solve the problem. I still need to read the binary data of the content so that I can turn it into a base64 encoded data-uri. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org