Interferenz zwischen zwei gleichzeitig geöffneten Dateien

Manni79 · July 10, 2023, 8:41am

Hallo zusammen,

ich habe einen Bug, den ich auch reproduzieren kann.

Ich habe zwei Dateien in PP offen, links und rechts nebeneinander, jeweils ein XML file geöffnet.
Links ist “alle Wertpapiere”, und rechts eine Watchlist gewählt
Wenn ich nun links das “+” und “neues Wertpapier” anklicke und ein Wertpapier hinzufüge, dann wird dieses auch rechts in der Watchlist zugefügt.

Seltsam ist, dieses Wertpapiert was nun auch rechts in der selektierten Watchliste hinzugekommen ist, ist rechts nicht zu finden, wenn man rechts “Alle Wertpapiere” auswählt. Sprich, es ist zwar rechts in der Watchliste zu sehen, wo es nicht sein sollte, aber eben nur in dieser einen Watchliste; und nicht unter “Alle Wertpapiere”.
Ich dachte bisher das geht gar nicht. Dementsprechend kann ich das Wertpapier rechts auch nur aus der Watchliste entfernen, aber nicht löschen (denn das geht nur unter “Alle Wertpapiere”).

Ich habe noch nicht getestet, ob das Wertpapier nur in der rechten Watchliste erscheint oder ob es dort auch im XML gespeichert ist.

Neuste PP Version.

Danke und beste Grüße!

OnkelDok · July 18, 2023, 7:10pm

Danke @Manni79 für deine ausführliche Beschreibung. Das Verhalten lässt sich nachstellen.
Die Wertpapiere sind dann auch in beiden Dateien hinzugefügt.

Nach erstem groben Duchsteppen durch den Code habe ich das Gefühl, dass die Ursache irgendwo innerhalb des Aufrufs broker.post liegt:

github.com

buchen/portfolio/blob/044c24f049189aadfa413ea77b6765578a55ec71/name.abuchen.portfolio.ui/src/name/abuchen/portfolio/ui/handlers/NewDomainElementHandler.java#L129


      
          private void openEditDialog(AbstractFinanceView view, Security newSecurity)
          {
              EditSecurityDialog editSecurityDialog = view.make(EditSecurityDialog.class, newSecurity);
          
              if (editSecurityDialog.open() == Window.OK)
              {
                  view.getClient().addSecurity(newSecurity);
                  view.getClient().markDirty();
                  new UpdateQuotesJob(view.getClient(), newSecurity).schedule();
          
                  broker.post(UIConstants.Event.Domain.SECURITY_CREATED, newSecurity);
              }
          }
          
          private void createNewCryptocurrency(AbstractFinanceView view)
          {
              try
              {
                  @SuppressWarnings("nls")
                  final Set<String> popularCoins = Set.of("bitcoin", "ethereum", "aave", "algorand", "bitcoin-cash",
                                  "cardano", "chainlink", "decentraland", "dogecoin", "litecoin", "polkadot", "matic-network",

Ich werde aber heute nicht dazu kommen tiefer in die entsprechenden Codestellen einzusteigen. Wenn ich die nächsten Tage Zeit finde, versuche ich mich mal noch genauer in die betreffenden Codestrukturen, Abläufe und Datenströme einzuarbeiten.
Vielleicht hat @AndreasB aber auch sofort die Problemstelle parat und löst das Problem.

EDIT
Ich habe nochmal geschaut: Durch das broker.post werden in beiden Portfolios diese Funktion aufgerufen:

github.com

buchen/portfolio/blob/044c24f049189aadfa413ea77b6765578a55ec71/name.abuchen.portfolio.ui/src/name/abuchen/portfolio/ui/views/SecurityListView.java#L384


      
          
          @Inject
          @Optional
          public void setup(@Named(UIConstants.Parameter.VIEW_PARAMETER) Predicate<Security> adhocFilter)
          {
              this.adhocFilter = adhocFilter;
          }
          
          @Inject
          @Optional
          public void onSecurityCreated(@UIEventTopic(UIConstants.Event.Domain.SECURITY_CREATED) Security newSecurity)
          {
              if (watchlist != null)
              {
                  watchlist.addSecurity(newSecurity);
                  getClient().touch();
              }
          
              setSecurityTableInput();
              securities.getTableViewer().setSelection(new StructuredSelection(newSecurity), true);
          }

In dieser erfolgt dann der Eintrag in die Watchliste.

Hier scheint der Fehler zu sein. Eigentlich dürfte das “Event” durch den Broker nur an das Portfolio gehen, was aktuell aktiv ist.
Ich vermute stark, dass @AndreasB da schnell eine Lösung finden kann. Ich bräuchte erstmal Einarbeitungszeit in die ganze Thematik.

Update:

Mit der nächsten Version sollte der Fehler behoben sein. Vielleicht wird sie schon in den nächsten Tagen verfügbar sein.

AndreasB · July 23, 2023, 8:06pm

Der Fix ist jetzt mit Version 0.64.5 veröffentlicht.

Habe ich schon mal gesagt, dass ich Open Source liebe? Ich habe den Fehler bei einem Umbau eingebaut, @OnkelDok hat ihn behoben.