[Libreoffice-commits] core.git: include/vcl padmin/source vcl/inc vcl/null vcl/unx

Caolán McNamara caolanm at redhat.com
Thu Mar 20 09:26:19 PDT 2014


 include/vcl/printerinfomanager.hxx             |   11 ---
 padmin/source/adddlg.cxx                       |   86 ++-----------------------
 padmin/source/adddlg.hxx                       |    5 -
 padmin/source/cmddlg.cxx                       |    2 
 padmin/source/padialog.cxx                     |   11 ---
 padmin/source/padialog.hrc                     |    7 --
 padmin/source/padialog.hxx                     |    3 
 padmin/source/padialog.src                     |   48 -------------
 padmin/source/prtsetup.cxx                     |    2 
 vcl/inc/cupsmgr.hxx                            |    2 
 vcl/null/printerinfomanager.cxx                |   20 -----
 vcl/unx/generic/printer/cupsmgr.cxx            |   10 --
 vcl/unx/generic/printer/printerinfomanager.cxx |   44 ------------
 13 files changed, 13 insertions(+), 238 deletions(-)

New commits:
commit aa4e31ab274cdcc53d715d923823adcdadb6be22
Author: Caolán McNamara <caolanm at redhat.com>
Date:   Thu Mar 20 15:51:06 2014 +0000

    disallow disabling cups
    
    which means that there is only one option left in
    "add a printer" in spadmin, that to add a pdf so remove
    the intermediate page
    
    Change-Id: I5f19b526b5da9307bec77c731f305280b0ebecc3

diff --git a/include/vcl/printerinfomanager.hxx b/include/vcl/printerinfomanager.hxx
index 6492f8c..5ccd366 100644
--- a/include/vcl/printerinfomanager.hxx
+++ b/include/vcl/printerinfomanager.hxx
@@ -112,8 +112,6 @@ protected:
     bool                              m_bUseJobPatch;
     OUString                     m_aSystemDefaultPaper;
 
-    bool                              m_bDisableCUPS;
-
     PrinterInfoManager( Type eType = Default );
 
     virtual void initialize();
@@ -195,21 +193,12 @@ public:
     // returns true on success
     virtual bool endSpool( const OUString& rPrinterName, const OUString& rJobTitle, FILE* pFile, const JobData& rDocumentJobData, bool bBanner, const OUString &rFaxNumber );
 
-    // for spadmin: whether adding or removing a printer is possible
-    virtual bool addOrRemovePossible() const;
-
     bool getUseIncludeFeature() const { return m_bUseIncludeFeature; }
     bool getUseJobPatch() const { return m_bUseJobPatch; }
 
     // check whether a printer's feature string contains a subfeature
     bool checkFeatureToken( const OUString& rPrinterName, const char* pToken ) const;
 
-    // set m_bDisableCUPS and update printer config
-    void setCUPSDisabled( bool );
-
-    // gets m_bDisableCUPS, initialized from printer config
-    bool isCUPSDisabled() const;
-
     virtual ~PrinterInfoManager();
 };
 
diff --git a/padmin/source/adddlg.cxx b/padmin/source/adddlg.cxx
index 2368a73..7e775df 100644
--- a/padmin/source/adddlg.cxx
+++ b/padmin/source/adddlg.cxx
@@ -46,41 +46,6 @@ APTabPage::APTabPage( AddPrinterDialog* pParent, const ResId& rResId )
 {
 }
 
-APChooseDevicePage::APChooseDevicePage( AddPrinterDialog* pParent ) :
-        APTabPage( pParent, PaResId( RID_ADDP_PAGE_CHOOSEDEV ) ),
-        m_aPrinterBtn( this, PaResId( RID_ADDP_CHDEV_BTN_PRINTER ) ),
-        m_aPDFBtn( this, PaResId( RID_ADDP_CHDEV_BTN_PDF ) ),
-        m_aOverTxt( this, PaResId( RID_ADDP_CHDEV_TXT_OVER ) )
-{
-    FreeResource();
-    m_aPrinterBtn.Check( true );
-    m_aPDFBtn.Check( false );
-    if( ! PrinterInfoManager::get().addOrRemovePossible() )
-    {
-        m_aPrinterBtn.Check( false );
-        m_aPrinterBtn.Enable( false );
-    }
-}
-
-APChooseDevicePage::~APChooseDevicePage()
-{
-}
-
-bool APChooseDevicePage::check()
-{
-    return true;
-}
-
-void APChooseDevicePage::fill( PrinterInfo& rInfo )
-{
-    if( m_aPDFBtn.IsChecked() )
-    {
-        rInfo.m_aFeatures = "pdf=";
-    }
-    else
-        rInfo.m_aFeatures = "";
-}
-
 APChooseDriverPage::APChooseDriverPage( AddPrinterDialog* pParent )
         : APTabPage( pParent, PaResId( RID_ADDP_PAGE_CHOOSEDRIVER ) ),
           m_aDriverTxt( this, PaResId( RID_ADDP_CHDRV_TXT_DRIVER ) ),
@@ -489,7 +454,6 @@ AddPrinterDialog::AddPrinterDialog( Window* pParent )
           m_aLine( this, PaResId( RID_ADDP_LINE ) ),
           m_aTitleImage( this, PaResId( RID_ADDP_CTRL_TITLE ) ),
           m_pCurrentPage( NULL ),
-          m_pChooseDevicePage( NULL ),
           m_pCommandPage( NULL ),
           m_pChooseDriverPage( NULL ),
           m_pNamePage( NULL ),
@@ -499,7 +463,8 @@ AddPrinterDialog::AddPrinterDialog( Window* pParent )
           m_pPdfCommandPage( NULL )
 {
     FreeResource();
-    m_pCurrentPage = m_pChooseDevicePage = new APChooseDevicePage( this );
+    m_pCurrentPage = m_pPdfDriverPage = new APPdfDriverPage( this );
+
     m_pCurrentPage->Show( true );
     m_aFinishPB.Enable( false );
     m_aPrevPB.Enable( false );
@@ -516,7 +481,6 @@ AddPrinterDialog::AddPrinterDialog( Window* pParent )
 
 AddPrinterDialog::~AddPrinterDialog()
 {
-    delete m_pChooseDevicePage;
     delete m_pChooseDriverPage;
     delete m_pNamePage;
     delete m_pCommandPage;
@@ -544,24 +508,7 @@ void AddPrinterDialog::DataChanged( const DataChangedEvent& rEv )
 void AddPrinterDialog::advance()
 {
     m_pCurrentPage->Show( false );
-    if( m_pCurrentPage == m_pChooseDevicePage )
-    {
-        if( m_pChooseDevicePage->isPrinter() )
-        {
-            if( ! m_pChooseDriverPage )
-                m_pChooseDriverPage = new APChooseDriverPage( this );
-            m_pCurrentPage = m_pChooseDriverPage;
-            m_aPrevPB.Enable( true );
-        }
-        else if( m_pChooseDevicePage->isPDF() )
-        {
-            if( ! m_pPdfDriverPage )
-                m_pPdfDriverPage = new APPdfDriverPage( this );
-            m_pCurrentPage = m_pPdfDriverPage;
-            m_aPrevPB.Enable( true );
-        }
-    }
-    else if( m_pCurrentPage == m_pChooseDriverPage )
+    if( m_pCurrentPage == m_pChooseDriverPage )
     {
         if( ! m_pCommandPage )
             m_pCommandPage = new APCommandPage( this, DeviceKind::Printer );
@@ -591,6 +538,7 @@ void AddPrinterDialog::advance()
                 m_pPdfCommandPage = new APCommandPage( this, DeviceKind::Pdf );
             m_pCurrentPage = m_pPdfCommandPage;
         }
+        m_aPrevPB.Enable( true );
     }
     else if( m_pCurrentPage == m_pPdfSelectDriverPage )
     {
@@ -614,12 +562,7 @@ void AddPrinterDialog::advance()
 void AddPrinterDialog::back()
 {
     m_pCurrentPage->Show( false );
-    if( m_pCurrentPage == m_pChooseDriverPage )
-    {
-        m_pCurrentPage = m_pChooseDevicePage;
-        m_aPrevPB.Enable( false );
-    }
-    else if( m_pCurrentPage == m_pNamePage )
+    if( m_pCurrentPage == m_pNamePage )
     {
         m_pCurrentPage = m_pCommandPage;
         m_aNextPB.Enable( true );
@@ -628,11 +571,6 @@ void AddPrinterDialog::back()
     {
         m_pCurrentPage = m_pChooseDriverPage;
     }
-    else if( m_pCurrentPage == m_pPdfDriverPage )
-    {
-        m_pCurrentPage = m_pChooseDevicePage;
-        m_aPrevPB.Enable( false );
-    }
     else if( m_pCurrentPage == m_pPdfSelectDriverPage )
     {
         m_pCurrentPage = m_pPdfDriverPage;
@@ -659,17 +597,9 @@ void AddPrinterDialog::addPrinter()
     {
         PrinterInfo aInfo( rManager.getPrinterInfo( m_aPrinter.m_aPrinterName ) );
         aInfo.m_aCommand = m_aPrinter.m_aCommand;
-        if( m_pChooseDevicePage->isPrinter() )
-        {
-            if( m_pNamePage->isDefault() )
-                rManager.setDefaultPrinter( m_aPrinter.m_aPrinterName );
-        }
-        else if( m_pChooseDevicePage->isPDF() )
-        {
-            OUString aPdf( "pdf=" );
-            aPdf += m_pPdfCommandPage->getPdfDir();
-            aInfo.m_aFeatures = aPdf;
-        }
+        OUString aPdf( "pdf=" );
+        aPdf += m_pPdfCommandPage->getPdfDir();
+        aInfo.m_aFeatures = aPdf;
         rManager.changePrinterInfo( m_aPrinter.m_aPrinterName, aInfo );
     }
 }
diff --git a/padmin/source/adddlg.hxx b/padmin/source/adddlg.hxx
index 486b19d..c0b54e0 100644
--- a/padmin/source/adddlg.hxx
+++ b/padmin/source/adddlg.hxx
@@ -53,16 +53,12 @@ public:
 
 class APChooseDevicePage : public APTabPage
 {
-    RadioButton             m_aPrinterBtn;
     RadioButton             m_aPDFBtn;
     FixedText               m_aOverTxt;
 public:
     APChooseDevicePage( AddPrinterDialog* pParent );
     ~APChooseDevicePage();
 
-    bool isPrinter() { return m_aPrinterBtn.IsChecked(); }
-    bool isPDF() { return m_aPDFBtn.IsChecked(); }
-
     virtual bool check();
     virtual void fill( ::psp::PrinterInfo& rInfo );
 };
@@ -180,7 +176,6 @@ class AddPrinterDialog : public ModalDialog
 
     APTabPage*              m_pCurrentPage;
 
-    APChooseDevicePage*     m_pChooseDevicePage;
     APCommandPage*          m_pCommandPage;
     APChooseDriverPage*     m_pChooseDriverPage;
     APNamePage*             m_pNamePage;
diff --git a/padmin/source/cmddlg.cxx b/padmin/source/cmddlg.cxx
index 8a4d72e..89908af 100644
--- a/padmin/source/cmddlg.cxx
+++ b/padmin/source/cmddlg.cxx
@@ -221,7 +221,7 @@ RTSCommandPage::RTSCommandPage( RTSDialog* pParent ) :
 {
     // configuring as printer is only sensible in default print system
     PrinterInfoManager& rMgr( PrinterInfoManager::get() );
-    if( rMgr.getType() == PrinterInfoManager::Default || rMgr.isCUPSDisabled() )
+    if( rMgr.getType() == PrinterInfoManager::Default )
         m_nPrinterEntry = m_aConfigureBox.InsertEntry( OUString( PaResId( RID_RTS_CMD_STR_CONFIGURE_PRINTER ) ) );
     else
         m_nPrinterEntry = ~0;
diff --git a/padmin/source/padialog.cxx b/padmin/source/padialog.cxx
index 9e8eff3..82a54bf 100644
--- a/padmin/source/padialog.cxx
+++ b/padmin/source/padialog.cxx
@@ -84,8 +84,6 @@ PADialog::PADialog( Window* pParent, sal_Bool /*bAdmin*/ ) :
         m_aCommand( this, PaResId( RID_PA_TXT_COMMAND_STRING ) ),
         m_aCommentTxt( this, PaResId( RID_PA_TXT_COMMENT ) ),
         m_aComment( this, PaResId( RID_PA_TXT_COMMENT_STRING ) ),
-        m_aCUPSFL( this, PaResId( RID_PA_FL_CUPSUSAGE ) ),
-        m_aCUPSCB( this, PaResId( RID_PA_CB_CUPSUSAGE ) ),
         m_aSepButtonFL( this, PaResId( RID_PA_FL_SEPBUTTON ) ),
         m_aAddPB( this, PaResId( RID_PA_BTN_ADD ) ),
         m_aCancelButton( this, PaResId( RID_PA_BTN_CANCEL ) ),
@@ -108,7 +106,6 @@ void PADialog::Init()
 {
     // #i79787# initially ensure printer discovery has ended
     m_rPIManager.checkPrintersChanged( true );
-    m_aCUPSCB.Check( m_rPIManager.isCUPSDisabled() );
 
     UpdateDevice();
     UpdateText();
@@ -124,7 +121,6 @@ void PADialog::Init()
     m_aTestPagePB.SetClickHdl( LINK( this, PADialog, ClickBtnHdl ) );
     m_aAddPB.SetClickHdl( LINK( this, PADialog, ClickBtnHdl ) );
     m_aDevicesLB.setDelPressedLink( LINK( this, PADialog, DelPressedHdl ) );
-    m_aCUPSCB.SetClickHdl( LINK( this, PADialog, ClickBtnHdl ) );
 
     // at this point no actual changes will be  written
     // but the write will have checked whether any writeable config exists
@@ -135,7 +131,6 @@ void PADialog::Init()
         m_aConfPB.Enable( false );
         m_aRenamePB.Enable( false );
         m_aStdPB.Enable( false );
-        m_aCUPSCB.Enable( false );
         ErrorBox aBox( GetParent(), WB_OK | WB_DEF_OK, OUString( PaResId( RID_ERR_NOWRITE ) ) );
         aBox.Execute();
     }
@@ -204,12 +199,6 @@ IMPL_LINK( PADialog, ClickBtnHdl, PushButton*, pButton )
         PrintTestPage();
     else if( pButton == &m_aAddPB )
         AddDevice();
-    else if( static_cast<Button*>(pButton) == &m_aCUPSCB )
-    {
-        m_rPIManager.setCUPSDisabled( m_aCUPSCB.IsChecked() );
-            UpdateDevice();
-            UpdateText();
-    }
 
     return 0;
 }
diff --git a/padmin/source/padialog.hrc b/padmin/source/padialog.hrc
index b9b095e..cda3f2d 100644
--- a/padmin/source/padialog.hrc
+++ b/padmin/source/padialog.hrc
@@ -43,8 +43,6 @@
 
 #define RID_PA_STR_RENAME                   21
 #define RID_PA_FL_SEPBUTTON                 22
-#define RID_PA_FL_CUPSUSAGE                 23
-#define RID_PA_CB_CUPSUSAGE                 23
 
 #define RID_PPDIMPORT_DLG          1004
 #define RID_PPDIMP_BTN_OK             1
@@ -106,11 +104,6 @@
 #define RID_ADDP_CTRL_TITLE                     6
 #define RID_ADDP_STR_TITLE                       127
 
-#define RID_ADDP_PAGE_CHOOSEDEV                 2001
-#define RID_ADDP_CHDEV_TXT_OVER                 1
-#define RID_ADDP_CHDEV_BTN_PRINTER              2
-#define RID_ADDP_CHDEV_BTN_PDF                  3
-
 #define RID_ADDP_PAGE_CHOOSEDRIVER              2002
 #define RID_ADDP_CHDRV_TXT_DRIVER               1
 #define RID_ADDP_CHDRV_BOX_DRIVER               2
diff --git a/padmin/source/padialog.hxx b/padmin/source/padialog.hxx
index e7620c2..0f38ffd 100644
--- a/padmin/source/padialog.hxx
+++ b/padmin/source/padialog.hxx
@@ -53,9 +53,6 @@ namespace padmin {
         FixedText                           m_aCommentTxt;
         FixedText                           m_aComment;
 
-        FixedLine                           m_aCUPSFL;
-        CheckBox                            m_aCUPSCB;
-
         FixedLine                           m_aSepButtonFL;
         PushButton                          m_aAddPB;
         CancelButton                        m_aCancelButton;
diff --git a/padmin/source/padialog.src b/padmin/source/padialog.src
index ad16e52..74b2d09 100644
--- a/padmin/source/padialog.src
+++ b/padmin/source/padialog.src
@@ -28,20 +28,6 @@ ModalDialog RID_PADIALOG
     Moveable = TRUE ;
     Closeable = TRUE ;
 
-    FixedLine RID_PA_FL_CUPSUSAGE
-    {
-        Pos = MAP_APPFONT( 6, 145 );
-        Size = MAP_APPFONT( 248, 8 );
-        Text [ en-US ] = "CUPS support";
-    };
-    CheckBox RID_PA_CB_CUPSUSAGE
-    {
-        HelpID = "padmin:CheckBox:RID_PADIALOG:RID_PA_CB_CUPSUSAGE";
-        Pos = MAP_APPFONT( 12, 158 );
-        Size = MAP_APPFONT( 168, 8 );
-        Text [ en-US ] = "Disable CUPS Support";
-    };
-
     FixedLine RID_PA_FL_SEPBUTTON
     {
         Pos = MAP_APPFONT( 0, 176 );
@@ -154,7 +140,7 @@ ModalDialog RID_PADIALOG
         HelpID = "padmin:PushButton:RID_PADIALOG:RID_PA_BTN_ADD";
         Pos = MAP_APPFONT( 5, 181 );
         Size = MAP_APPFONT( 70, 12 );
-        Text [ en-US ] = "New Printer...";
+        Text [ en-US ] = "New PDF Converter...";
     };
 
     String RID_PA_STR_DEFPRT
@@ -506,38 +492,6 @@ TabPage RID_ADDP_PAGE_CHOOSEDRIVER
     };
 };
 
-TabPage RID_ADDP_PAGE_CHOOSEDEV
-{
-    HelpID = "padmin:TabPage:RID_ADDP_PAGE_CHOOSEDEV";
-    Hide = TRUE;
-    Pos = MAP_APPFONT( 0, 30 );
-    Size = MAP_APPFONT( 240, 110 );
-    String RID_ADDP_STR_TITLE
-    {
-        Text [ en-US ] = "Choose a device type";
-    };
-    FixedText RID_ADDP_CHDEV_TXT_OVER
-    {
-        Pos = MAP_APPFONT( 40, 25 );
-        Size = MAP_APPFONT( 180, 8 );
-        Text [ en-US ] = "Do you want to";
-    };
-    RadioButton RID_ADDP_CHDEV_BTN_PRINTER
-    {
-        HelpID = "padmin:RadioButton:RID_ADDP_PAGE_CHOOSEDEV:RID_ADDP_CHDEV_BTN_PRINTER";
-        Pos = MAP_APPFONT ( 40, 40 );
-        Size = MAP_APPFONT( 180, 10 );
-        Text [ en-US ] = "Add a ~printer";
-    };
-    RadioButton RID_ADDP_CHDEV_BTN_PDF
-    {
-        HelpID = "padmin:RadioButton:RID_ADDP_PAGE_CHOOSEDEV:RID_ADDP_CHDEV_BTN_PDF";
-        Pos = MAP_APPFONT ( 40, 50 );
-        Size = MAP_APPFONT( 180, 10 );
-        Text [ en-US ] = "Connect a P~DF converter";
-    };
-};
-
 TabPage RID_ADDP_PAGE_NAME
 {
     HelpID = "padmin:TabPage:RID_ADDP_PAGE_NAME";
diff --git a/padmin/source/prtsetup.cxx b/padmin/source/prtsetup.cxx
index 4082de6..39911b5 100644
--- a/padmin/source/prtsetup.cxx
+++ b/padmin/source/prtsetup.cxx
@@ -96,7 +96,7 @@ RTSDialog::RTSDialog( const PrinterInfo& rJobData, const OUString& rPrinter, boo
         m_pTabControl->RemovePage(m_pTabControl->GetPageId("other"));
         m_pTabControl->RemovePage(m_pTabControl->GetPageId("command"));
     }
-    else if( m_aJobData.m_aDriverName.startsWith("CUPS:") && ! PrinterInfoManager::get().isCUPSDisabled() )
+    else if (m_aJobData.m_aDriverName.startsWith("CUPS:"))
     {
         // command page makes no sense for CUPS printers
         m_pTabControl->RemovePage(m_pTabControl->GetPageId("command"));
diff --git a/vcl/inc/cupsmgr.hxx b/vcl/inc/cupsmgr.hxx
index 8454c31..cfbd774 100644
--- a/vcl/inc/cupsmgr.hxx
+++ b/vcl/inc/cupsmgr.hxx
@@ -92,8 +92,6 @@ public:
     virtual bool removePrinter( const OUString& rPrinterName, bool bCheckOnly = false );
     virtual bool writePrinterConfig();
     virtual bool setDefaultPrinter( const OUString& rPrinterName );
-
-    virtual bool addOrRemovePossible() const;
 };
 
 } // namespace psp
diff --git a/vcl/null/printerinfomanager.cxx b/vcl/null/printerinfomanager.cxx
index d102f0c..d787219 100644
--- a/vcl/null/printerinfomanager.cxx
+++ b/vcl/null/printerinfomanager.cxx
@@ -46,11 +46,6 @@ PrinterInfoManager::PrinterInfoManager( Type eType ) :
     m_bUseIncludeFeature( false ),
     m_bUseJobPatch( true ),
     m_aSystemDefaultPaper( "A4" ),
-#ifdef LIBO_HEADLESS
-    m_bDisableCUPS( true )
-#else
-    m_bDisableCUPS( false )
-#endif
 {
     // initSystemDefaultPaper();
 }
@@ -70,16 +65,6 @@ void PrinterInfoManager::initialize()
     // ???
 }
 
-bool PrinterInfoManager::isCUPSDisabled() const
-{
-    return m_bDisableCUPS;
-}
-
-void PrinterInfoManager::setCUPSDisabled( bool /* bDisable */ )
-{
-    // cups is already disabled in config so do nothing
-}
-
 void PrinterInfoManager::listPrinters( ::std::list< OUString >& rList ) const
 {
     rList.clear();
@@ -117,11 +102,6 @@ bool PrinterInfoManager::setDefaultPrinter( const OUString& /* rPrinterName */ )
     return false;
 }
 
-bool PrinterInfoManager::addOrRemovePossible() const
-{
-    return false;
-}
-
 void PrinterInfoManager::getSystemPrintCommands( std::list< OUString >& /* rCommands */ )
 {
 
diff --git a/vcl/unx/generic/printer/cupsmgr.cxx b/vcl/unx/generic/printer/cupsmgr.cxx
index 3627cd7..9c7af07 100644
--- a/vcl/unx/generic/printer/cupsmgr.cxx
+++ b/vcl/unx/generic/printer/cupsmgr.cxx
@@ -256,9 +256,6 @@ void CUPSManager::initialize()
     if( ! (m_nDests && m_pDests ) )
         return;
 
-    if( isCUPSDisabled() )
-        return;
-
     // check for CUPS server(?) > 1.2
     // since there is no API to query, check for options that were
     // introduced in dests with 1.2
@@ -424,7 +421,7 @@ const PPDParser* CUPSManager::createCUPSParser( const OUString& rPrinter )
 
     if( m_aCUPSMutex.tryToAcquire() )
     {
-        if( m_nDests && m_pDests && ! isCUPSDisabled() )
+        if (m_nDests && m_pDests)
         {
             boost::unordered_map< OUString, int, OUStringHash >::iterator dest_it =
             m_aCUPSDestMap.find( aPrinter );
@@ -830,11 +827,6 @@ bool CUPSManager::writePrinterConfig()
     return PrinterInfoManager::writePrinterConfig();
 }
 
-bool CUPSManager::addOrRemovePossible() const
-{
-    return (m_nDests && m_pDests && ! isCUPSDisabled())? false : PrinterInfoManager::addOrRemovePossible();
-}
-
 const char* CUPSManager::authenticateUser( const char* /*pIn*/ )
 {
     const char* pRet = NULL;
diff --git a/vcl/unx/generic/printer/printerinfomanager.cxx b/vcl/unx/generic/printer/printerinfomanager.cxx
index 01d6ce6..35116c1 100644
--- a/vcl/unx/generic/printer/printerinfomanager.cxx
+++ b/vcl/unx/generic/printer/printerinfomanager.cxx
@@ -117,16 +117,13 @@ PrinterInfoManager::PrinterInfoManager( Type eType ) :
     m_eType( eType ),
     m_bUseIncludeFeature( false ),
     m_bUseJobPatch( true ),
-    m_aSystemDefaultPaper( "A4" ),
-    m_bDisableCUPS( false )
+    m_aSystemDefaultPaper( "A4" )
 {
     if( eType == Default )
         m_pQueueInfo = new SystemQueueInfo();
     initSystemDefaultPaper();
 }
 
-
-
 PrinterInfoManager::~PrinterInfoManager()
 {
     delete m_pQueueInfo;
@@ -135,26 +132,6 @@ PrinterInfoManager::~PrinterInfoManager()
     #endif
 }
 
-
-
-bool PrinterInfoManager::isCUPSDisabled() const
-{
-    return m_bDisableCUPS;
-}
-
-
-
-void PrinterInfoManager::setCUPSDisabled( bool bDisable )
-{
-    m_bDisableCUPS = bDisable;
-    writePrinterConfig();
-    // actually we know the printers changed
-    // however this triggers reinitialization the right way
-    checkPrintersChanged( true );
-}
-
-
-
 void PrinterInfoManager::initSystemDefaultPaper()
 {
     m_aSystemDefaultPaper = OStringToOUString(
@@ -231,7 +208,6 @@ void PrinterInfoManager::initialize()
     // need a parser for the PPDContext. generic printer should do.
     m_aGlobalDefaults.m_pParser = PPDParser::getParser( OUString( "SGENPRT" ) );
     m_aGlobalDefaults.m_aContext.setParser( m_aGlobalDefaults.m_pParser );
-    m_bDisableCUPS = false;
 
     if( ! m_aGlobalDefaults.m_pParser )
     {
@@ -291,15 +267,6 @@ void PrinterInfoManager::initialize()
             if (!aValue.isEmpty())
                 m_aGlobalDefaults.m_nPDFDevice = aValue.toInt32();
 
-            aValue = aConfig.ReadKey( "DisableCUPS" );
-            if (!aValue.isEmpty())
-            {
-                if (aValue.equals("1") || aValue.equalsIgnoreAsciiCase("true"))
-                    m_bDisableCUPS = true;
-                else
-                    m_bDisableCUPS = false;
-            }
-
             // get the PPDContext of global JobData
             for( int nKey = 0; nKey < aConfig.GetKeyCount(); ++nKey )
             {
@@ -656,7 +623,6 @@ bool PrinterInfoManager::writePrinterConfig()
 
     Config* pGlobal = files.begin()->second;
     pGlobal->SetGroup( GLOBAL_DEFAULTS_GROUP );
-    pGlobal->WriteKey( "DisableCUPS", m_bDisableCUPS ? "true" : "false" );
 
     ::boost::unordered_map< OUString, Printer, OUStringHash >::iterator it;
     for( it = m_aPrinters.begin(); it != m_aPrinters.end(); ++it )
@@ -900,14 +866,6 @@ bool PrinterInfoManager::setDefaultPrinter( const OUString& rPrinterName )
     return bSuccess;
 }
 
-
-bool PrinterInfoManager::addOrRemovePossible() const
-{
-    return true;
-}
-
-
-
 void PrinterInfoManager::getSystemPrintCommands( std::list< OUString >& rCommands )
 {
     if( m_pQueueInfo && m_pQueueInfo->hasChanged() )


More information about the Libreoffice-commits mailing list