ShowStorage() funktioniert nicht mehr

noci · Jul 3rd 2023

red51 ich hab da ein Phenomän mit den neu gesetzten Kisten seit ca. dem 14.06.2023 warscheinlich mit/nach dem Update 0.6.5: Plugin API Hotfix 0.6.5.1

Die Funktion player.showStorage scheint für die Kiste die danach desetzt worden sind, Funktionieren nicht mehr:

Code

Create new JNIEnv  (handle: 2413587882256, thread: 28)
[Java] Storage ID [1843676]
[Java] Global ID [1843676]
[Java] nicht Akiv
Object chest1 (1843676) change status 0 -> 1
Create new JNIEnv  (handle: 2413587850976, thread: 19)
[Java] Mouse Button[1] 
[DB] Saved 1 players in database (0 ms)
[DB] Saved 1 storages in database (0 ms)
Object chest1 (1843676) change status 1 -> 0
[Java] ToolsAPI Command |/ch| [/ch]
[Java] net.risingworld.api.objects.Player@4ce041ce> /ch
[Java] Mimic Command |/ch| [/ch]
[Java] Storage ID [1843676] Open
[Java] UnityBudelLoader Command |/ch| [/ch]
[DB] Saved 1 chunks in database (0 ms)
Object leathertrunk (30173788) change status 1 -> 0
[DB] Saved 1 players in database (0 ms)
[DB] Saved 2 storages in database (0 ms)
[Java] Storage ID [30173788]
[Java] Global ID [30173788]
[Java] nicht Akiv
Object leathertrunk (30173788) change status 0 -> 1
[DB] Saved 5 npcs in database (0 ms)
[Java] Mouse Button[1] 
Object leathertrunk (30173788) change status 1 -> 0
[Java] ToolsAPI Command |/ch| [/ch]
[Java] net.risingworld.api.objects.Player@4ce041ce> /ch
[Java] Mimic Command |/ch| [/ch]
[Java] Storage ID [30173788] Open
[Java] UnityBudelLoader Command |/ch| [/ch]
[WARNING] [14:01:22] Storage 13396572 (object: False) does not exist...

Display More

Die besagte Kiste 30173788 wird als 13396572 in der Datenbank gesuch, die es aber nicht gibt

In der DB wird die 30173788 benutzt

Wir haben auf jeden fall einige von den neuen Kisten bei den es Funktioniert und welche die heute gesetzt wurden bei den es nicht mehr Funktioniert

red51 · Jul 3rd 2023

Hey, ich habe den Beitrag mal in einen separaten Thread verschoben, da es ja in erster Linie um die API geht

Also ich konnte das Problem jetzt auf Anhieb leider nicht reproduzieren Grundsätzlich sollte es für die API egal sein, ob die Kiste vor oder nach dem Update platziert wurde. Solange man die Kiste im Spiel noch öffnen kann und sie ihren korrekten Inhalt hat, sollte damit alles i.O. sein (hinter den Kulissen verwendet die API eigentlich auch nur die gleichen Routinen, um auf den Kisteninhalt zuzugreifen).

Kannst du mir sonst ggf. einmal den verantwortlichen Code dazu zeigen?

noci · Jul 3rd 2023

Ja, ich komme da öfter durcheinander, wo ich da am Besten schreibe

Java

    @EventMethod
    public void onPlayerStorageAccessEvent(PlayerStorageAccessEvent event) {
        System.out.println("Storage ID ["+event.getStorageID()+"]");
        System.out.println("Global ID ["+event.getGlobalID()+"]");
        Player player = event.getPlayer();
        //player.setAttribute(pa_Last_Storage_ID, event.getStorageID());
        player.setAttribute(pa_Last_Storage_ID, event.getGlobalID());
        player.setAttribute(pa_Last_Storage_CX, event.getChunkPositionX());
        player.setAttribute(pa_Last_Storage_CY, event.getChunkPositionY());
        player.setAttribute(pa_Last_Storage_CZ, event.getChunkPositionZ());
        //event.setCancelled(true);
        //event.getPlayer().showStorage(1843676);
        //InventoryLayer/chestContainer
        UIElement playerUIElement = (UIElement)player.getAttribute(pa_UI_Panel);
//        if ((Long)player.getAttribute(pa_SelectStorage_ID)!=null)
        if ( (Long)player.getAttribute(pa_SelectStorage_ID)!=null && (Long)player.getAttribute(pa_SelectStorage_ID)==event.getStorageID() ) {
            System.out.println("Da und Aktiv");
            playerUIElement.style.backgroundImage.set(star1);
            playerUIElement.updateStyle();
        }else{
            System.out.println("nicht Akiv");
            playerUIElement.style.backgroundImage.set(star0);
            playerUIElement.updateStyle();
        }
    }/**/
    
    @EventMethod
    public void onPlayerUIElementClickEvent (PlayerUIElementClickEvent event){
        Player user = event.getPlayer();
        UIElement uiElement = event.getUIElement();
        Arrays.asList(Server.getAllPlayers()).forEach((player) -> {
            if (player == user) {
                System.out.println("Mouse Button["+event.getMouseButton()+"] ");
                //UIElement playerUIElement = (UIElement)player.getAttribute(pa_UI_Panel);
                //UILabel playerUILabel = (UILabel)player.getAttribute(pa_UI_Panel);
                UIElement playerUIElement = (UIElement)player.getAttribute(pa_UI_Panel);
                if ( player.getAttribute(pa_UI_Panel)!=null && playerUIElement==uiElement ){
                    switch (event.getMouseButton()) {
                        case 0 -> {
                            //Server.
                            if ( (Long)player.getAttribute(pa_SelectStorage_ID)!=null ) {
                                //WorldItem storage = World.getItem((Long)player.getAttribute(pa_SelectStorage_ID));
                                //MetaObject metaObject = World.getMetaObject( (Long)player.getAttribute(pa_SelectStorage_ID) );
                                ObjectElement object = World.getObject((Long)player.getAttribute(pa_SelectStorage_ID), (Integer)player.getAttribute(pa_SelectStorage_CX), (Integer)player.getAttribute(pa_SelectStorage_CY), (Integer)player.getAttribute(pa_SelectStorage_CZ));
                                Objects.ObjectDefinition definition = object.getDefinition();
                                player.sendTextMessage(definition.relateditem);
                                player.sendTextMessage(definition.name);
                                player.sendTextMessage(definition.type.name());
                                if (definition.type.equals(Type.Storage)){
                                    //player.sendTextMessage("Storage Aktion");
                                }
                                player.sendTextMessage("\n");
                            }
                        }
                        case 1 -> {
                            //playerUILabel.setText(playerUILabel.getText()+" Click");
                            player.setAttribute(pa_SelectStorage_ID, player.getAttribute(pa_Last_Storage_ID));
                            player.setAttribute(pa_SelectStorage_CX, player.getAttribute(pa_Last_Storage_CX));
                            player.setAttribute(pa_SelectStorage_CY, player.getAttribute(pa_Last_Storage_CY));
                            player.setAttribute(pa_SelectStorage_CZ, player.getAttribute(pa_Last_Storage_CZ));
                            //playerUILabel.setText(Long.toString((Long)player.getAttribute(pa_SelectStorage_ID)));
                            playerUIElement.style.backgroundImage.set(star1);
                            playerUIElement.updateStyle();
                        }
                        case 2 -> {
                            player.deleteAttribute(pa_SelectStorage_ID);
                            //playerUILabel.setText("Auswählen");
                            playerUIElement.style.backgroundImage.set(star0);
                            playerUIElement.updateStyle();
                        }
                    }
                }
            }
        });
    }
    
    @EventMethod
    public void onCommand(PlayerCommandEvent event){
        Player player = event.getPlayer();
        String command = event.getCommand();
        String[] parameter = command.split(" ");
        System.out.println("Mimic "+"Command |"+command+"| "+"["+parameter[0]+"]");
        if(command.equals("/ch")){
            if (player.getAttribute(pa_SelectStorage_ID)!=null){
                System.out.println("Storage ID ["+player.getAttribute(pa_SelectStorage_ID)+"] Open");
                player.showStorage(((Long)player.getAttribute(pa_SelectStorage_ID)));
                
            }
        }
    }

Display More

Das ist der Kernteil der Geschichte
Also die ID's werden eigendlich Koreckt übergeben

noci · Jul 3rd 2023

Jetzt habe ich mal eine "Neue Welt" erstellt und in der Klappt das ohne Probleme mit den neu gesetzten Kisten

Könnte das beim Konvertieren Passiert sein?

Kann ich da was Reparieren?

red51 · Jul 4th 2023

Oha, da scheint es ein Problem mit den IDs zu geben Die Kisten ID wird falsch serialisiert (der 64 bit Wert wird auf zwei 32 bit Werte aufgeteilt)... dummerweise funktioniert es meistens, wenn aber die IDs zu hoch werden, kommt Blödsinn dabei raus... darum funktioniert es bei dir in einer neuen Welt (und deshalb hat es vorher wohl auch zufällig funktioniert)...

Sorry für den Ärger! Das werden wir mit dem kommenden Hotfix beheben Das ist wie gesagt kein Problem mit den Kisten, damit ist alles i.O., es liegt stattdessen am Spiel, was damit momentan nicht korrekt umgeht^^ Nach dem Hotfix sollte also alles wieder laufen.

noci · Jul 4th 2023

Quote from red51

Sorry für den Ärger!

Kein ding, wehrend der Entwicklung, sehe ich Fehler finden das als Sport an
Desto mehr ich finde deso schneller ist die Basis Fertig

Ludy · Jul 4th 2023

Quote from red51

Oha, da scheint es ein Problem mit den IDs zu geben Die Kisten ID wird falsch serialisiert (der 64 bit Wert wird auf zwei 32 bit Werte aufgeteilt)... dummerweise funktioniert es meistens, wenn aber die IDs zu hoch werden, kommt Blödsinn dabei raus... darum funktioniert es bei dir in einer neuen Welt (und deshalb hat es vorher wohl auch zufällig funktioniert)...

Sorry für den Ärger! Das werden wir mit dem kommenden Hotfix beheben Das ist wie gesagt kein Problem mit den Kisten, damit ist alles i.O., es liegt stattdessen am Spiel, was damit momentan nicht korrekt umgeht^^ Nach dem Hotfix sollte also alles wieder laufen.

Freu wir müssen nicht noch mal neu anfangen (ich war mir sicher wir (ich) haben die welt getötet)
danke für die schnelle entwarung
war schon am koffer packen "aufbruch in eine neue welt wurde verschoben "

Participate now!