Kaydet (Commit) d0525e56 authored tarafından Katarina Behrens's avatar Katarina Behrens Kaydeden (comit) Andras Timar

Related tdf#91362: disable paper size & orientation selection

Unless 'Use only papersize from printer prefs' is toggled on,
whatever the user sets here doesn't have any effect on the actual
printing (page settings from the document are used instead). It is
misleading to have the user believe otherwise.

It was really challenging to have this one toggle get all the way
down through several layers of abstraction, though ...

Change-Id: If240084ca23b3946a92882a6ee47dbc161b3b86c
Reviewed-on: https://gerrit.libreoffice.org/16822Reviewed-by: 's avatarMiklos Vajna <vmiklos@collabora.co.uk>
Tested-by: 's avatarMiklos Vajna <vmiklos@collabora.co.uk>
üst d8722a7e
......@@ -320,7 +320,7 @@ public:
bool SetJobSetup( const JobSetup& rSetup );
const JobSetup& GetJobSetup() const { return maJobSetup; }
bool Setup( vcl::Window* pWindow = NULL );
bool Setup( vcl::Window* pWindow = NULL, bool bPapersizeFromSetup = false );
bool SetPrinterProps( const Printer* pPrinter );
// SetPrinterOptions is used internally only now
......
......@@ -50,6 +50,7 @@ struct PrinterInfo : JobData
// a list of special features separated by ',' not used by psprint
// but assigned from the outside (currently for "fax","pdf=","autoqueue","external_dialog")
OUString m_aFeatures;
bool m_bPapersizeFromSetup;
PrinterInfo() :
JobData()
......
......@@ -560,6 +560,7 @@ bool PspSalInfoPrinter::Setup( SalFrame* pFrame, ImplJobSetup* pJobSetup )
SetData( ~0, pJobSetup );
JobData::constructFromStreamBuffer( pJobSetup->mpDriverData, pJobSetup->mnDriverDataLen, aInfo );
}
aInfo.m_bPapersizeFromSetup = pJobSetup->mbPapersizeFromSetup;
if (SetupPrinterDriver(aInfo))
{
......
......@@ -256,6 +256,14 @@ void RTSPaperPage::update()
m_pSlotText->Enable( false );
m_pSlotBox->Enable( false );
}
// disable those, unless user wants to use papersize from printer prefs
// as they have no influence on what's going to be printed anyway
if (!m_pParent->m_aJobData.m_bPapersizeFromSetup)
{
m_pPaperBox->Enable( false );
m_pOrientBox->Enable( false );
}
}
IMPL_LINK( RTSPaperPage, SelectHdl, ListBox*, pBox )
......
......@@ -43,6 +43,7 @@ struct ImplJobSetup
long mnPaperHeight; // paper height (100th mm)
sal_uIntPtr mnDriverDataLen; // length of system specific data
sal_uInt8* mpDriverData; // system specific data (will be streamed a byte block)
bool mbPapersizeFromSetup;
std::unordered_map< OUString, OUString, OUStringHash > maValueMap;
ImplJobSetup();
......
......@@ -61,6 +61,7 @@ ImplJobSetup::ImplJobSetup()
mnPaperHeight = 0;
mnDriverDataLen = 0;
mpDriverData = NULL;
mbPapersizeFromSetup = false;
}
ImplJobSetup::ImplJobSetup( const ImplJobSetup& rJobSetup ) :
......@@ -83,6 +84,7 @@ ImplJobSetup::ImplJobSetup( const ImplJobSetup& rJobSetup ) :
}
else
mpDriverData = NULL;
mbPapersizeFromSetup = rJobSetup.mbPapersizeFromSetup;
maValueMap = rJobSetup.maValueMap;
}
......
......@@ -1133,7 +1133,7 @@ bool Printer::SetJobSetup( const JobSetup& rSetup )
return false;
}
bool Printer::Setup( vcl::Window* pWindow )
bool Printer::Setup( vcl::Window* pWindow, bool bPapersizeFromSetup )
{
if ( IsDisplayPrinter() )
return false;
......@@ -1142,6 +1142,8 @@ bool Printer::Setup( vcl::Window* pWindow )
return false;
JobSetup aJobSetup = maJobSetup;
ImplJobSetup* pData = aJobSetup.ImplGetData();
pData->mbPapersizeFromSetup = bPapersizeFromSetup;
SalFrame* pFrame;
if ( !pWindow )
pWindow = ImplGetDefaultWindow();
......@@ -1153,7 +1155,7 @@ bool Printer::Setup( vcl::Window* pWindow )
ImplSVData* pSVData = ImplGetSVData();
pSVData->maAppData.mnModalMode++;
nImplSysDialog++;
bool bSetup = mpInfoPrinter->Setup( pFrame, aJobSetup.ImplGetData() );
bool bSetup = mpInfoPrinter->Setup( pFrame, pData );
pSVData->maAppData.mnModalMode--;
nImplSysDialog--;
if ( bSetup )
......
......@@ -825,7 +825,7 @@ bool PrinterController::setupPrinter( vcl::Window* i_pParent )
}
// call driver setup
bRet = mpImplData->mxPrinter->Setup( i_pParent );
bRet = mpImplData->mxPrinter->Setup( i_pParent, getPapersizeFromSetup() );
Size aNewPaperSize(mpImplData->mxPrinter->GetPaperSize());
if (bRet)
{
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment