pharo-vcs / iceberg

Iceberg is the main toolset for handling VCS in Pharo.
MIT License
133 stars 84 forks source link

Push window does not honor selected remote #1831

Closed guillep closed 2 months ago

guillep commented 3 months ago

I click on push, the window is opened with a default remote selected (which is ok to me, I want to push there)

imagen

I click on Push, and!

imagen

I have a permission denied. Of course, It tried to push to another remote and not the one selected!

Moreover, if I change remote (select the second remote, then select back the first one in the list), everything is ok.

Finally, the list of commits shown to push are not the ones to push to the selected remote but to the other one!

It's like the shown remote is not the selected remote.

Ducasse commented 3 months ago

This is the same bug we faced with Quentin Moutté

guillep commented 3 months ago

I chased this down a bit.

The problem is that inside the drop down, the dropdown items are all equal (they respond true to =) systematically. This seems to happen because the drop list items have not their display block set, so they return an asString of the model.

This means there are two issues:

guillep commented 3 months ago

The answer to the first question is that it is the droplist that has the display block

initializePresenters

    remoteList := self newDropList
        display: [ :each | each shortDescription ];
        yourself.
...

So, is the display block in the item required/useful?

guillep commented 2 months ago

Fixed in P12 and P13 in Spec