Improved: General refactoring and code improvements, package
org.apache.ofbiz.common.qrcode.
Thanks Julian Leichert for reporting and providing the patches.
@@ -53,8 +53,12 @@ public class QRCodeEvents {
Map<String, Object> parameters = UtilHttp.getParameterMap(request);
String message = (String) parameters.get("message");
GenericValue userLogin = (GenericValue) request.getAttribute("userLogin");
- if (userLogin == null) userLogin = (GenericValue) session.getAttribute("userLogin");
- if (userLogin == null) userLogin = (GenericValue) session.getAttribute("autoUserLogin");
+ if (userLogin == null) {
+ userLogin = (GenericValue) session.getAttribute("userLogin");
+ }
+ if (userLogin == null) {
+ userLogin = (GenericValue) session.getAttribute("autoUserLogin");
+ }
Locale locale = UtilHttp.getLocale(request);
if (UtilValidate.isEmpty(message)) {
@@ -138,11 +142,7 @@ public class QRCodeEvents {
request.setAttribute("_ERROR_MESSAGE_", errMsg);
return "error";
}
- } catch (IOException e) {
- String errMsg = UtilProperties.getMessage("QRCodeUiLabels", "ErrorGenerateQRCode", new Object[] { e.getMessage() }, locale);
- request.setAttribute("_ERROR_MESSAGE_", errMsg);
- return "error";
- } catch (GenericServiceException e) {
+ } catch (IOException | GenericServiceException e) {
String errMsg = UtilProperties.getMessage("QRCodeUiLabels", "ErrorGenerateQRCode", new Object[] { e.getMessage() }, locale);
request.setAttribute("_ERROR_MESSAGE_", errMsg);
return "error";
