<div dir="ltr"><div>Thanks, Chris and Bert.</div>I think I found the reason I had problems.<div>If you change instance variables but not any methods you get the methodListMenu:  from MCOperationsBrowser but not the one  from MCCodeTool.</div><div><br></div><div>There is only &#39;browse origins&#39; in the MCOperationsBrowser&gt;&gt;methodListMenu:</div><div><br></div><div>So the fix must be to fix MCCodeTool&gt;&gt;methodListMenu: to at least give option to &#39;browse full&#39; so one can get to the class.</div><div><br></div><div>Best,</div><div>Karl</div><div><div><br></div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 24, 2016 at 9:27 PM, Chris Muller <span dir="ltr">&lt;<a href="mailto:asqueaker@gmail.com" target="_blank">asqueaker@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi Karl, mcModel is for the history function, the code shouldn&#39;t be<br>
getting there for a regular browse class.  Are you sure you didn&#39;t<br>
select &quot;browse mc history&quot; or &quot;browse mc origin&quot;?<br>
<br>
PS -- I still plan to try to fix the mc history function again..<br>
<div><div><br>
On Tue, May 24, 2016 at 12:31 PM, karl ramberg &lt;<a href="mailto:karlramberg@gmail.com" target="_blank">karlramberg@gmail.com</a>&gt; wrote:<br>
&gt; Ok, it only fails if MCMethodDefinition&gt;&gt;mcModel is nil in<br>
&gt; MCWorkingCopyBrowser&gt;&gt;mcModelFor: aClassOrMethodReference<br>
&gt;<br>
&gt; Once you have the MCMethodDefinition&gt;&gt;mcModel installed it will work the<br>
&gt; next time, so my fix is still necessary<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; Best,<br>
&gt; Karl<br>
&gt;<br>
&gt; On Tue, May 24, 2016 at 6:41 PM, karl ramberg &lt;<a href="mailto:karlramberg@gmail.com" target="_blank">karlramberg@gmail.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Hm, weird.<br>
&gt;&gt; I removed the fix I did and now it works...<br>
&gt;&gt;<br>
&gt;&gt; Not sure what was going on<br>
&gt;&gt;<br>
&gt;&gt; Best,<br>
&gt;&gt; Karl<br>
&gt;&gt;<br>
&gt;&gt; On Tue, May 24, 2016 at 6:33 PM, Bert Freudenberg &lt;<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>&gt;<br>
&gt;&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Which menu item are you trying? You wrote “can open a browser from the<br>
&gt;&gt;&gt; context menu” and for me both cmd-b and the context menu work for opening a<br>
&gt;&gt;&gt; System Browser.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; - Bert -<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On 24.05.2016, at 18:27, karl ramberg &lt;<a href="mailto:karlramberg@gmail.com" target="_blank">karlramberg@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I know packageInfo is not the same as workingCopy but I&#39;m not sure how to<br>
&gt;&gt;&gt; reference it from the merge browser.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; How can this be working in your image ? packageInfo is not implemented<br>
&gt;&gt;&gt; for MCMethodDefinition...<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; It&#39;s called from<br>
&gt;&gt;&gt; MCWorkingCopyBrowser&gt;&gt;mcModelFor: aClassOrMethodReference<br>
&gt;&gt;&gt; ^ aClassOrMethodReference mcModel ifNil:<br>
&gt;&gt;&gt; [ | pkgName rep | (UIManager confirm: &#39;Okay to add historical repository<br>
&gt;&gt;&gt; &#39; , (rep := MCHttpRepository trunkBackup) description , &#39; to &#39; , (pkgName :=<br>
&gt;&gt;&gt; aClassOrMethodReference packageInfo packageName) , &#39;?&#39;) ifTrue:<br>
&gt;&gt;&gt; [ (MCWorkingCopy forPackageNamed: pkgName) repositoryGroup addRepository:<br>
&gt;&gt;&gt; rep.<br>
&gt;&gt;&gt; aClassOrMethodReference mcModel ] ]<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Best,<br>
&gt;&gt;&gt; Karl<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On Tue, May 24, 2016 at 5:56 PM, Bert Freudenberg &lt;<a href="mailto:bert@freudenbergs.de" target="_blank">bert@freudenbergs.de</a>&gt;<br>
&gt;&gt;&gt; wrote:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; On 24.05.2016, at 17:17, karl ramberg &lt;<a href="mailto:karlramberg@gmail.com" target="_blank">karlramberg@gmail.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt;&gt; &gt; I&#39;m pretty sure this is a proper fix so the MCMergeBrowser can open a<br>
&gt;&gt;&gt;&gt; &gt; browser from the context menu:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; This appears to work fine for me even without that change.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; &gt; MCMethodDefinition&gt;&gt;packageInfo<br>
&gt;&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt;&gt; &gt;        ^ self workingCopy! !<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; … and a WorkingCopy most definitely is not the same as a PackageInfo ;)<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; - Bert -<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
<br>
</div></div></blockquote></div><br></div></div></div></div>