<div dir="ltr">Hi Lee,<div><br></div><div><div style="font-family:arial,sans-serif;font-size:13px"><div>> > 2) Create a CellProfilerApp plugin</div><div>> </div><div>> It looks like I have to do this</div><div><br></div><div>It is nice for other reasons, too: then CellProfiler will have a "presence" in the context, so if you run e.g. the SystemInformation command [1], you will see CellProfiler in the output! :-)</div></div><div style="font-family:arial,sans-serif;font-size:13px"><br></div><div>Also, if you didn't see my blog post on quitting a couple of months, that may be worth a read:</div><div><br></div><div><a href="http://imagej.net/2014-07-11_-_Fiji_won't_quit!#Other_complications">http://imagej.net/2014-07-11_-_Fiji_won't_quit!#Other_complications</a><br></div><div><br></div><div>Especially the "Other complications" section which explains that there are multiple very different code paths for quitting.</div><div><br></div><div>So depending on what code CellProfiler is invoking -- A) ij.ImageJ#quit() vs. B) org.scijava.Context#dispose() vs. C) publication of an AppQuitEvent -- you may experience different behavior.</div><div><br></div><div>It has been long enough now that all the details do not spring to mind, but please reply back with any further questions and we can iron out the details. I'd _like_ to believe that the quit architecture is as elegant & configurable as possible now, but feel free to prove me wrong if there are any roadblocks.</div><div><br></div><div>Regards,</div><div>Curtis</div></div><div><br></div><div>[1] <a href="https://github.com/scijava/scijava-plugins-commands/blob/scijava-plugins-commands-0.1.7/src/main/java/org/scijava/plugins/commands/debug/SystemInformation.java#L95-L116">https://github.com/scijava/scijava-plugins-commands/blob/scijava-plugins-commands-0.1.7/src/main/java/org/scijava/plugins/commands/debug/SystemInformation.java#L95-L116</a></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 5, 2014 at 11:44 AM, Lee Kamentsky <span dir="ltr"><<a href="mailto:leek@broadinstitute.org" target="_blank">leek@broadinstitute.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks, Curtis:<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 5, 2014 at 12:33 PM, Curtis Rueden <span dir="ltr"><<a href="mailto:ctrueden@wisc.edu" target="_blank">ctrueden@wisc.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Lee,<span class=""><div><br></div><div><div>So, there are two potential ways I would suggest to achieve your goal:<br></div></div><div><br></div><div>1) Set the OptionsMisc exitWhenQuitting to false. This is handy if you don't need a lot of control over exactly what happens when quit() is called, but you just want to stop the System.exit(0) from firing.</div><div><br></div></span></div></blockquote><div>I have code to do this, but in the upgrade to the latest, perhaps I'm not running it and setting exitWhenQuitting to false. </div><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div></div><div>2) Create a CellProfilerApp plugin and set it to a higher priority than ImageJ. This solution makes the most sense to me because it gives you more flexibility over the behavior of a couple other operations, too: about() and prefs().</div></div></blockquote></span><div>It looks like I have to do this, because we don't want to lose the context. I think that will work out OK.</div><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div><a href="https://github.com/scijava/scijava-common/blob/scijava-common-2.35.0/src/main/java/org/scijava/app/App.java" target="_blank">https://github.com/scijava/scijava-common/blob/scijava-common-2.35.0/src/main/java/org/scijava/app/App.java</a><br></div><div><br></div><div>Regards,</div><div>Curtis</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 5, 2014 at 10:23 AM, Lee Kamentsky <span dir="ltr"><<a href="mailto:leek@broadinstitute.org" target="_blank">leek@broadinstitute.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Oops never mind, what I tried actually worked. AppQuitEvent.consume() did the trick. Go figure - you guys really threw the whole kitchen sink into it, didn't you?<div><br><div><div><span style="font-family:'courier new',monospace">@Plugin(type = Service.class, priority = Priority.HIGH_PRIORITY)</span><br></div><div><font face="courier new, monospace">public class CellProfilerAppService extends DefaultAppService {</font></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> </span><span style="font-family:'courier new',monospace">static boolean canQuit = false;</span><br></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> </span><span style="font-family:'courier new',monospace">public static void allowQuit() {</span><br></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>canQuit = true;</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>}</font></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> </span><span style="font-family:'courier new',monospace">public static void preventQuit() {</span><br></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>canQuit = false;</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>}</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>@EventHandler</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>public void onEvent(final AppQuitEvent event) {</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>if (canQuit) {</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>super.onEvent(event);</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>} else {</font></div><div><span style="font-family:'courier new',monospace;white-space:pre-wrap"> </span><span style="font-family:'courier new',monospace">final UIService uiService = getContext().getService(</span><span style="font-family:'courier new',monospace">UIService.class);</span><br></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>final LogService logService = getContext().getService(LogService.class);</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>if (uiService.isVisible()) {</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>UserInterface ui = uiService.getDefaultUI();</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>logService.info("Quit action: hide the application frame");</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>ui.getApplicationFrame().setVisible(false);</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>} else {</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>logService.info("Quit action: do nothing");</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>}</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>event.consume();</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>}</font></div><div><font face="courier new, monospace"><span style="white-space:pre-wrap"> </span>}</font></div><div><font face="courier new, monospace"><br></font></div><div><font face="courier new, monospace">}</font></div></div><div><br></div></div></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 5, 2014 at 11:15 AM, Lee Kamentsky <span dir="ltr"><<a href="mailto:leek@broadinstitute.org" target="_blank">leek@broadinstitute.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div>I had a class (<a href="https://github.com/CellProfiler/CellProfiler/blob/master/java/src/main/java/org/cellprofiler/ijutils/CellProfilerAppEventService.java" target="_blank">https://github.com/CellProfiler/CellProfiler/blob/master/java/src/main/java/org/cellprofiler/ijutils/CellProfilerAppEventService.java</a>) that implemented the deprecated AppEventService whose purpose was to prevent ImageJ from quitting if a user closed its window by overriding AppEventService.quit(). Quitting is pretty devastating for CellProfiler since the process closes when the user's probable intent was to hide the window.</div><div><br></div><div>I'm hoping someone can give me a hint about how to do it now - I thought I'd cheat by asking instead of figuring it out myself.</div><div><br></div><div>Thanks in advance,</div><div>--Lee</div></div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
ImageJ-devel mailing list<br>
<a href="mailto:ImageJ-devel@imagej.net" target="_blank">ImageJ-devel@imagej.net</a><br>
<a href="http://imagej.net/mailman/listinfo/imagej-devel" target="_blank">http://imagej.net/mailman/listinfo/imagej-devel</a><br>
<br></blockquote></div><br></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div><br></div>