To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=49631
------- Additional comments from [EMAIL PROTECTED] Sun Jun 5 23:55:54 -0700
2005 -------
Thanks for your comments and patience!
There are basically three things we need to get done:
1) Pick up the configuration value from the desktop
2) Transfer and implement the desktop value in OOo
3) When the desktop configuration value changes, update OOo
I guess updating the value would involve more things, but it would be great if
the first two steps are ok to start with. I have tried to take in all the
comments and am attaching the modified patch - it does not add in a gconf
callback :-) and I have added in a check for the same setting in
vcl_sal::PrinterUpdate::update() not to check for printer changes each time.
The other options where printer initialization happens seem to be at:
vcl/source/gdi/print.cxx: 539
SalPrinterQueueInfo* Printer::ImplGetQueueInfo( ... )
sfx2/source/view/printer.cxx: 307
SfxPrinter::SfxPrinter( SfxItemSet* pTheOptions,
const JobSetup& rTheOrigJobSetup ) :
Printer ( rTheOrigJobSetup.GetPrinterName() ),
pOptions ( pTheOptions )
{
pImpl = new SfxPrinter_Impl;
bKnown = GetName() == rTheOrigJobSetup.GetPrinterName();
if ( bKnown )
SetJobSetup( rTheOrigJobSetup );
}
sfx2/source/doc/sfxbasemodel.cxx:
XPrintable interface: getPrinter(), setPrinter(), print()
Could you please review if this patch would be acceptable for taking care of the
first two steps or please provide input on what would need to be done further ?
Regarding the third step of updation, I will look into it further but of what I
had seen earlier, I had been thinking that broadcaster.cxx seemed to be
returning NULL for the getCppuType for the gconf layer and succeeding for the
Options dialog change, does that means there are no listeners attached to it ?
Also, Application::MergeSystemSettings() is what is used in
http://go-ooo.org/lxr/source/graphics/svx/source/dialog/optgdlg.cxx#842,
http://go-ooo.org/lxr/source/util/svtools/source/config/accessibilityoptions.cxx#295
already which is what I had tried as well ? Is it really not acceptable ?
This is the one major option that needs to be introduced into OOo, and there is
no pre-existing option that it can be integrated with. The others can be read
into the particular existing configuration options. Please let us know what you
think, thanks!
---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
