svn commit: r1818848 - /ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

svn commit: r1818848 - /ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java

mbrohl
Author: mbrohl
Date: Wed Dec 20 20:10:15 2017
New Revision: 1818848

URL: http://svn.apache.org/viewvc?rev=1818848&view=rev
Log:
Improved: Logging: change System.out/.err/.. println to Debug.log...
in PdfSurveyServices.java.
(OFBIZ-10046)

Thanks Kyra Pritzel-Hentley for reporting and providing the patch.

Modified:
    ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java

Modified: ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java?rev=1818848&r1=1818847&r2=1818848&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java (original)
+++ ofbiz/ofbiz-framework/trunk/applications/content/src/main/java/org/apache/ofbiz/content/survey/PdfSurveyServices.java Wed Dec 20 20:10:15 2017
@@ -298,7 +298,7 @@ public class PdfSurveyServices {
             }
 
         } catch (DocumentException | GeneralException | IOException e) {
-            System.err.println(e.getMessage());
+            Debug.logError(e, module);
             return ServiceUtil.returnError(e.getMessage());
         }
 
@@ -348,7 +348,7 @@ public class PdfSurveyServices {
             ByteBuffer outByteBuffer = ByteBuffer.wrap(baos.toByteArray());
             results.put("outByteBuffer", outByteBuffer);
         } catch (DocumentException | IOException | GeneralException e) {
-            System.err.println(e.getMessage());
+            Debug.logError(e, module);
             results = ServiceUtil.returnError(e.getMessage());
         }
         return results;
@@ -414,7 +414,7 @@ public class PdfSurveyServices {
             ByteBuffer outByteBuffer = ByteBuffer.wrap(baos.toByteArray());
             results.put("outByteBuffer", outByteBuffer);
         } catch (GenericEntityException | DocumentException e) {
-            System.err.println(e.getMessage());
+            Debug.logError(e, module);
             results = ServiceUtil.returnError(e.getMessage());
         }
 
@@ -439,7 +439,7 @@ public class PdfSurveyServices {
             }
             results.put("questionsAndAnswers", qAndA);
         } catch (GenericEntityException e) {
-            System.err.println(e.getMessage());
+            Debug.logError(e, module);
             results = ServiceUtil.returnError(e.getMessage());
         }
 
@@ -505,7 +505,7 @@ public class PdfSurveyServices {
                 acroFieldMap.put(fieldName, value);
             }
         } catch (GenericEntityException e) {
-            System.err.println(e.getMessage());
+            Debug.logError(e, module);
             return ServiceUtil.returnError(e.getMessage());
         }