Hi Lauren,

Thanks for submitting this.

Can you say something about how to reproduce the problem for this fix?

Here is what I did:

- Open an Monticello browser
- Add a new repository (just a local file based repository for this test)
- Open a second Monticello browser
- In the second Monticello browser, remove the repository that I created in the first one
- Go back to the first Monticello browser, and select the repository that I deleted in other one

The produces the subscript out of bounds error that you have addressed.
However the fix seems to produce some other side effects that I
do not understand.

Is this the same scenario that you are addressing?

Thanks :-)
Dave


On Sat, Jan 15, 2022 at 03:27:16AM +0000, Lauren Pullen wrote:
> Hi List,
> 
> On 1/15/22 02:20, commits at source.squeak.org wrote:
> > A new version of Monticello was added to project The Inbox:
> > http://source.squeak.org/inbox/Monticello-lrnp.757.mcz
> 
> This one is mine.
> 
> I have a long-running Monticello window open in the HomeProject.  I had
> made then deleted a directory repository at the bottom of the list, but
> it ended up retained in the UI but not the model.*
> 
> This patch catches the out-of-bounds error to refresh the list so you
> can try again.
> 
> Would this change be more appropriate in PluggableListMorph? or would it
> be too difficult to detect in general?  I'm not very familiar with
> ToolBuilder.
> 
> * I'm not entirely sure how it happened to test further.
>