Changeset 871


Ignore:
Timestamp:
08/22/09 17:45:37 (3 years ago)
Author:
jharrop
Message:

Get rid of System.out.println (mostly).

Location:
trunk/docx4j/src/main/java/org/docx4j
Files:
25 edited

Legend:

Unmodified
Added
Removed

  • trunk/docx4j/src/main/java/org/docx4j/JcrNodeMapper/AlfrescoJcrNodeMapper.java

    r296 r871  
    6868                 
    6969                log.info("getting {http://www.alfresco.org/model/content/1.0}content");  
    70                 System.out.println("getting {http://www.alfresco.org/model/content/1.0}content");  
    7170                // Jackrabbit's jcr:data = Alfresco's cm:content 
    7271                 

  • trunk/docx4j/src/main/java/org/docx4j/convert/in/FlatOpcXmlImporter.java

    r854 r871  
    531531                boolean prettyprint = true; 
    532532                 
    533                 System.out.println(  
     533                log.debug(  
    534534                                org.docx4j.XmlUtils. 
    535535                                        marshaltoString(result, suppressDeclaration, prettyprint,  

  • trunk/docx4j/src/main/java/org/docx4j/diff/Differencer.java

    r840 r871  
    12201220                                                } 
    12211221                                        } catch (XMLStreamException e) { 
    1222                                                 System.out.println("Issue at element: " + reader.getLocalName() + "\n"); 
    1223                                                 e.printStackTrace(); 
     1222                                                log.error("Issue at element: " + reader.getLocalName() + "\n", e); 
    12241223                                                throw e; 
    12251224                                        } 

  • trunk/docx4j/src/main/java/org/docx4j/fonts/BestMatchingMapper.java

    r713 r871  
    170170                u2.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    171171 
    172                 System.out.println("unmarshalling fonts.substitutions \n\n" );                                                                   
     172                log.info("unmarshalling fonts.substitutions" );                                                                  
    173173                // Get the xml file 
    174174                java.io.InputStream is2 = null; 
    … …  
    714714                         
    715715                                if (fopPanose == null ) { 
    716                                         System.out.println(fontName + " .. lacks Panose!");                                      
     716                                        log.warn(fontName + " .. lacks Panose!");                                        
    717717                                } else if (fopPanose!=null ) { 
    718                                         System.out.println(fontName + " .. " + fopPanose); 
     718                                        log.debug(fontName + " .. " + fopPanose); 
    719719                                } 
    720720//                                      long pd = fopPanose.difference(nfontInfo.getPanose().getPanoseArray()); 

  • trunk/docx4j/src/main/java/org/docx4j/fonts/Mapper.java

    r715 r871  
    2525 
    2626import org.apache.log4j.Logger; 
    27 import org.docx4j.XmlUtils; 
    2827 
    2928/** 

  • trunk/docx4j/src/main/java/org/docx4j/jaxb/NamespacePrefixMapper.java

    r779 r871  
    6868     *      value will be ignored and the system will generate one. 
    6969     */ 
    70     public String getPreferredPrefix(String namespaceUri, String suggestion, boolean requirePrefix) {     
     70    public String getPreferredPrefix(String namespaceUri, String suggestion, boolean requirePrefix)  
     71    // Implement the interface 
     72    { 
     73         
     74        return getPreferredPrefixStatic(namespaceUri, suggestion, requirePrefix); 
     75         
     76    } 
     77     
     78     
     79    protected static String getPreferredPrefixStatic(String namespaceUri, String suggestion, boolean requirePrefix) {     
    7180         
    7281        if (namespaceUri.equals(Namespaces.NS_WORD12)) { 
    … …  
    215224        public String getNamespaceURI(String prefix) { 
    216225                 
     226                return getNamespaceURIStatic(prefix); 
     227        } 
     228         
     229        protected static String getNamespaceURIStatic(String prefix) { 
     230         
    217231                if (prefix.equals("w"))   
    218232                        return Namespaces.NS_WORD12; 

  • trunk/docx4j/src/main/java/org/docx4j/jaxb/NamespacePrefixMapperRelationshipsPart.java

    r788 r871  
    2222 
    2323public class NamespacePrefixMapperRelationshipsPart extends com.sun.xml.bind.marshaller.NamespacePrefixMapper { 
    24         // Must use 'internal' for Java 6 
    25  
    2624         
    2725    /** 
    … …  
    6159     *      value will be ignored and the system will generate one. 
    6260     */ 
    63     public String getPreferredPrefix(String namespaceUri, String suggestion, boolean requirePrefix) { 
     61    public String getPreferredPrefix(String namespaceUri, String suggestion, boolean requirePrefix)  
     62    // Implement the interface     
     63    { 
     64 
     65        return getPreferredPrefixStatic( namespaceUri,  suggestion,  requirePrefix); 
     66    } 
     67     
     68    protected static String getPreferredPrefixStatic(String namespaceUri, String suggestion, boolean requirePrefix) { 
    6469     
    6570//      if (namespaceUri.equals("http://schemas.openxmlformats.org/wordprocessingml/2006/main")) { 

  • trunk/docx4j/src/main/java/org/docx4j/model/listnumbering/Emulator.java

    r757 r871  
    166166                         
    167167                        if (numPr==null) { 
    168                                 System.out.println("Couldn't get NumPr from " +  pStyleVal); 
    169                                 System.out.println( 
     168                                log.warn("Couldn't get NumPr from " +  pStyleVal); 
     169                                log.debug( 
    170170                                                org.docx4j.XmlUtils.marshaltoString(style, true, true) 
    171171                                                ); 

  • trunk/docx4j/src/main/java/org/docx4j/model/listnumbering/NumberFormatLowerLetter.java

    r750 r871  
    11package org.docx4j.model.listnumbering; 
    22 
     3import org.apache.log4j.Logger; 
     4 
    35public class NumberFormatLowerLetter extends NumberFormat { 
     6         
     7        protected static Logger log = Logger.getLogger(NumberFormatLowerLetter.class); 
    48         
    59        public String format( int in ) { 
    … …  
    812                StringBuilder out = new StringBuilder(); 
    913                 
    10                 for( int i=0; i < str.length() ; i++ ) { 
    11                          
    12                          
    13                          int dig = Character.digit(str.charAt(i), 36); 
    14                           
    15                          
    16                         char cdig = Character.forDigit(dig+9, 36); 
    17                         
    18                          
    19                         System.out.println(str.charAt(i) + " --> " + cdig); 
    20                          
     14                for (int i = 0; i < str.length(); i++) { 
     15                        int dig = Character.digit(str.charAt(i), 36); 
     16                        char cdig = Character.forDigit(dig + 9, 36); 
     17                        log.debug(str.charAt(i) + " --> " + cdig); 
    2118                        out.append(cdig); 
    22                          
    23                 } 
    24                  
    25                 return out.toString(); 
    26                  
     19                }                
     20                return out.toString();           
    2721        } 
    2822         

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/Base.java

    r668 r871  
    8282                        return contentType.toString(); 
    8383                } else { 
    84                         System.out.println("WARNING: content type was null. " + 
     84                        log.warn("WARNING: content type was null. " + 
    8585                                        "You should set this before adding the part."); 
    8686                        return null; 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/packages/WordprocessingMLPackage.java

    r854 r871  
    4141import org.docx4j.XmlUtils; 
    4242import org.docx4j.convert.out.flatOpcXml.FlatOpcXmlCreator; 
    43 import org.docx4j.fonts.BestMatchingMapper; 
    4443import org.docx4j.fonts.IdentityPlusMapper; 
    4544import org.docx4j.fonts.Mapper; 
    … …  
    293292                java.util.Map fontsInUse = this.getMainDocumentPart().fontsInUse(); 
    294293                 
    295                 if ( fm instanceof BestMatchingMapper ) { 
     294                //if ( fm instanceof BestMatchingMapper ) { 
     295                if ( fm.getClass().getName().equals("org.docx4j.fonts.BestMatchingMapper") ) { 
    296296                         
    297297                         

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/DocPropsCorePart.java

    r302 r871  
    102102                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    103103 
    104                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     104                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    105105                                                 
    106106                        jaxbElement = u.unmarshal( is ); 
    107107                         
    108108                         
    109                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     109                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    110110 
    111111                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/DocPropsCustomPart.java

    r542 r871  
    127127                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    128128 
    129                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     129                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    130130                                                 
    131131                        jaxbElement = u.unmarshal( is ); 
    132132                         
    133133                         
    134                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     134                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    135135 
    136136                } catch (Exception e ) { 
    … …  
    149149                for (org.docx4j.docProps.custom.Properties.Property prop: customProps.getProperty() ) { 
    150150                         
    151                         System.out.println(prop.getName()); 
     151                        log.info(prop.getName()); 
    152152                         
    153153                        if (prop.getName().equals(propName)) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/DocPropsExtendedPart.java

    r609 r871  
    114114                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    115115 
    116                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     116                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    117117                                                 
    118118                        jaxbElement = u.unmarshal( is ); 
    119119                         
    120120                         
    121                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     121                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    122122 
    123123                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/Dom4jXmlPart.java

    r667 r871  
    4040 *  the Part should extend this class.    
    4141 * */ 
     42@Deprecated 
    4243public abstract class Dom4jXmlPart extends Part { 
    4344         

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/ThemePart.java

    r302 r871  
    9898                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    9999 
    100                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     100                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    101101                                                 
    102102                        jaxbElement = u.unmarshal( is ); 
    103103                         
    104104                         
    105                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     105                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    106106 
    107107                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/BinaryPartAbstractImage.java

    r684 r871  
    523523                  Dimension dPx = size.getDimensionPx(); 
    524524 
    525                   System.out.println(info.getOriginalURI() + " " + info.getMimeType()  
     525                  log.debug(info.getOriginalURI() + " " + info.getMimeType()  
    526526                                  + " " + Math.round(dPx.getWidth()) +"x" + Math.round(dPx.getHeight())); 
    527527                                   
    528                   System.out.println("Resolution:" + Math.round(size.getDpiHorizontal()) + "x" + Math.round(size.getDpiVertical()) ); 
    529                   System.out.println("Print size: " + Math.round(dPt.getWidth()/72) + "\" x" + Math.round(dPt.getHeight()/72)+"\"" );  
     528                  log.debug("Resolution:" + Math.round(size.getDpiHorizontal()) + "x" + Math.round(size.getDpiVertical()) ); 
     529                  log.debug("Print size: " + Math.round(dPt.getWidth()/72) + "\" x" + Math.round(dPt.getHeight()/72)+"\"" );  
    530530                 
    531531        } 
    … …  
    605605         */ 
    606606                 
    607          System.out.println("Start ImageMagick..."); 
     607         log.info("Start ImageMagick..."); 
    608608         Process p = Runtime.getRuntime().exec("imconvert -density " + density + " -units PixelsPerInch - png:-");   
    609609          
    … …  
    635635                 copy2(is, new BufferedOutputStream(p.getOutputStream())); 
    636636                 p.getOutputStream().close(); 
    637                  System.out.println("Image copied..."); 
     637                 log.debug("Image copied..."); 
    638638         } catch (IOException ioe) { 
    639639                  
    … …  
    644644          
    645645         if (p.waitFor()!=0) { 
    646           System.err.println("Error"); 
     646          log.error("Error"); 
    647647         } 
    648          System.out.println("End Process..."); 
     648         log.debug("End Process..."); 
    649649        } 
    650650 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/FontTablePart.java

    r695 r871  
    9696                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    9797 
    98                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     98                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    9999                                                 
    100100                        jaxbElement = u.unmarshal( is ); 
    101101                         
    102102                         
    103                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     103                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    104104 
    105105                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/FooterPart.java

    r545 r871  
    8989                                u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    9090 
    91                                 System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     91                                log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    9292                                                         
    9393                                jaxbElement = u.unmarshal( is ); 
    9494                                                                 
    95                                 System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     95                                log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    9696 
    9797                        } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/HeaderPart.java

    r545 r871  
    8989                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    9090 
    91                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     91                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    9292                                                 
    9393                        jaxbElement = u.unmarshal( is ); 
    9494                                                         
    95                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     95                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    9696 
    9797                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/NumberingDefinitionsPart.java

    r757 r871  
    310310                        jaxbElement = u.unmarshal( is ); 
    311311                         
    312                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     312                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    313313 
    314314                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/StyleDefinitionsPart.java

    r757 r871  
    9595                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    9696 
    97                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     97                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    9898                                                 
    9999                        jaxbElement = u.unmarshal( is ); 
    100100                         
    101                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     101                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    102102 
    103103                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/relationships/RelationshipsPart.java

    r869 r871  
    756756                        u.setEventHandler(new org.docx4j.jaxb.JaxbValidationEventHandler()); 
    757757 
    758                         System.out.println("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                     
     758                        log.info("unmarshalling " + this.getClass().getName() + " \n\n" );                                                                       
    759759                                                 
    760760                        jaxbElement = u.unmarshal( is ); 
    761761                         
    762762                         
    763                         System.out.println("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                  
     763                        log.info("\n\n" + this.getClass().getName() + " unmarshalled \n\n" );                                                                    
    764764 
    765765                } catch (Exception e ) { 

  • trunk/docx4j/src/main/java/org/docx4j/utils/ResourceUtils.java

    r305 r871  
    2121package org.docx4j.utils; 
    2222 
     23import org.apache.log4j.Logger; 
     24 
    2325public class ResourceUtils { 
     26         
     27        protected static Logger log = Logger.getLogger(ResourceUtils.class);     
    2428         
    2529    public static java.io.InputStream getResource(String filename) throws java.io.IOException 
    … …  
    3034         
    3135        if (url == null ) { 
    32                 System.out.println("Couldn't get resource!!"); 
     36                log.error("Couldn't get resource: " + filename); 
    3337        } 
    3438         

  • trunk/docx4j/src/main/java/org/docx4j/wml/P.java

    r809 r871  
    678678        List<Object> children = getParagraphContent(); 
    679679         
    680 //      System.out.println("p.toString"); 
     680//      log.debug("p.toString"); 
    681681         
    682682                for (Object o : children ) {                                     
    683 //                      System.out.println("  " + o.getClass().getName() ); 
     683//                      log.debug("  " + o.getClass().getName() ); 
    684684                        if ( o instanceof org.docx4j.wml.R) { 
    685 //                      System.out.println("Hit R"); 
     685//                      log.debug("Hit R"); 
    686686                                org.docx4j.wml.R  run = (org.docx4j.wml.R)o; 
    687687                        List runContent = run.getRunContent(); 
    … …  
    690690                                                // TODO - unmarshall directly to Text. 
    691691                                                if ( ((JAXBElement)o2).getDeclaredType().getName().equals("org.docx4j.wml.Text") ) { 
    692 //                                              System.out.println("Found Text"); 
     692//                                              log.debug("Found Text"); 
    693693                                                        org.docx4j.wml.Text t = (org.docx4j.wml.Text)((JAXBElement)o2).getValue(); 
    694694                                                        result.append( t.getValue() );                                   
    695695                                                } 
    696696                                        } else { 
    697 //                                      System.out.println(o2.getClass().getName());                                             
     697//                                      log.debug(o2.getClass().getName());                                              
    698698                                        } 
    699699                                } 
Note: See TracChangeset for help on using the changeset viewer.