<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 6, 2013 at 3:37 PM, Caolán McNamara <span dir="ltr"><<a href="mailto:caolanm@redhat.com" target="_blank">caolanm@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Mon, 2013-02-04 at 11:18 +0100, Jan Holesovsky wrote:<br>
> The code responsible for all this seems unnecessarily complicated to me,<br>
> so I am sorry if my quick read is actually misleading - but in general,<br>
> you don't want to move parts of sw/ to vcl, more so if it meant<br>
> including headers from sw in vcl.<br>
<br>
</div>Yeah, I wanted to give this a proper look, but never carved out the<br>
time, short answer is that what we want to happen is roughly<br>
a) remove that implicit store of settings in the writer code where it<br>
happens as a side effect and get it called explicitly in some store<br>
options configuration e.g.<br>
in PrintDialog::storeToSettings have a maPController->storeUIOptions();<br>
call and that maPController fundamentally comes from<br>
new SfxPrinterController in sfx2/source/view/viewprn.cxx and takes a<br>
SfxViewShell as an argument so it should be possible somewhere around<br>
there to add a virtual storeUIOptions to the vcl::PrinterController,<br>
implement it in SfxPrinterController and propagate (here its the usual<br>
frustrating haze of "shells") the task to yet-another app-specific<br>
virtual store method and move the writer implicit-save code into that<br>
method instead.<br>
<span class="HOEnZb"><font color="#888888"><br>
C.<br>
<br>
</font></span></blockquote></div><div class="gmail_extra"><br></div><div class="gmail_extra"><br></div>Thanks to both of you for the pointers. I'm hesitant to tackle this now as it seems well beyond my ability but....we'll see if I can find a friend or family member to do it with me.</div>
<div class="gmail_extra" style><br></div><div class="gmail_extra" style>Best Regards,</div><div class="gmail_extra" style>Joel</div><div class="gmail_extra"><br clear="all"><div><br></div>-- <br><div dir="ltr"><b><font face="garamond, serif" size="4">Joel Madero</font></b><div>
<font face="garamond, serif">LibreOffice QA Volunteer</font></div><div><font face="garamond, serif"><a href="mailto:jmadero.dev@gmail.com" target="_blank">jmadero.dev@gmail.com</a></font></div><div><br></div></div>
</div></div>