[ 
https://issues.apache.org/jira/browse/PDFBOX-974?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Hewson updated PDFBOX-974:
-------------------------------
    Description: 
Microsoft Os are mostly case insensitive.
If a printerName is given to PrintPDF, it looks for the provided string in a 
case sensitive manner ( line around 115 ).

I made a little correction to the code to allow non case sensitive search on 
Windows os :
I changed :
{code}
                      if(printService[i].getName().indexOf(printerName) != -1)
                      {
                          printJob.setPrintService(printService[i]);
                          printerFound = true;
                      }
{code}
to :
{code}
                    if (isWindows())
                    {
                      
if(printService[i].getName().toLowerCase().indexOf(printerName.toLowerCase()) 
!= -1)
                      {
                          printJob.setPrintService(printService[i]);
                          printerFound = true;
                      }
                    }
                    else
                    {
                      if(printService[i].getName().indexOf(printerName) != -1)
                      {
                          printJob.setPrintService(printService[i]);
                          printerFound = true;
                      }
                    }
{code}
and added the method :
{code}
    public static boolean isWindows()
    {
        String os = System.getProperty("os.name").toLowerCase();
        return (os.indexOf( "win" ) >= 0);
    }
{code}
I'm not sure it's the best way to do it, but at least, it works !


  was:
Microsoft Os are mostly case insensitive.
If a printerName is given to PrintPDF, it looks for the provided string in a 
case sensitive manner ( line around 115 ).

I made a little correction to the code to allow non case sensitive search on 
Windows os :
I changed :

                      if(printService[i].getName().indexOf(printerName) != -1)
                      {
                          printJob.setPrintService(printService[i]);
                          printerFound = true;
                      }
to :
                    if (isWindows())
                    {
                      
if(printService[i].getName().toLowerCase().indexOf(printerName.toLowerCase()) 
!= -1)
                      {
                          printJob.setPrintService(printService[i]);
                          printerFound = true;
                      }
                    }
                    else
                    {
                      if(printService[i].getName().indexOf(printerName) != -1)
                      {
                          printJob.setPrintService(printService[i]);
                          printerFound = true;
                      }
                    }
and added the method :
    public static boolean isWindows()
    {
        String os = System.getProperty("os.name").toLowerCase();
        return (os.indexOf( "win" ) >= 0);
    }

I'm not sure it's the best way to do it, but at least, it works !



> PrintPDF should not be case sensitive against printer name on Windows
> ---------------------------------------------------------------------
>
>                 Key: PDFBOX-974
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-974
>             Project: PDFBox
>          Issue Type: Wish
>          Components: Utilities
>    Affects Versions: 1.5.0
>         Environment: Windows XP/ JRE 1.6.0_24
>            Reporter: Christophe Borivant
>
> Microsoft Os are mostly case insensitive.
> If a printerName is given to PrintPDF, it looks for the provided string in a 
> case sensitive manner ( line around 115 ).
> I made a little correction to the code to allow non case sensitive search on 
> Windows os :
> I changed :
> {code}
>                       if(printService[i].getName().indexOf(printerName) != -1)
>                       {
>                           printJob.setPrintService(printService[i]);
>                           printerFound = true;
>                       }
> {code}
> to :
> {code}
>                     if (isWindows())
>                     {
>                       
> if(printService[i].getName().toLowerCase().indexOf(printerName.toLowerCase()) 
> != -1)
>                       {
>                           printJob.setPrintService(printService[i]);
>                           printerFound = true;
>                       }
>                     }
>                     else
>                     {
>                       if(printService[i].getName().indexOf(printerName) != -1)
>                       {
>                           printJob.setPrintService(printService[i]);
>                           printerFound = true;
>                       }
>                     }
> {code}
> and added the method :
> {code}
>     public static boolean isWindows()
>     {
>         String os = System.getProperty("os.name").toLowerCase();
>         return (os.indexOf( "win" ) >= 0);
>     }
> {code}
> I'm not sure it's the best way to do it, but at least, it works !



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to