BBCode + eigene Tags = doppelte und dreifache Einträge

Das ist ein Bug der Erweiterung. Der Autor ist darüber informiert, hat es aber leider noch nicht geschafft, diesen Fehler zu beheben. S. dazu Update: BBCode 0.4.1.

N'Abend,

ich hole das Thema mal wieder aus der Versenkung. Mir ging der Fehler schon immer ziemlich auf den Senkel und heute kam ich auf die Idee, den Code näher anzuschauen. Mit meinen spärlichen Kenntnissen konnte ich in der Datei
chrome\content\bbcode\bbcode.js (in der bbcode.jar) zumindest die Funktion ausfindig machen, die fürs Darstellen der Menüeinträge zuständig ist:

PopulateCustomImages: function(event){
    var theNumber = this.getPrefBBCodeCompose().GetIntPref('bbcode.tags.custom.number');
    var bbc_MainMenu = document.getElementById("bbcode_ct_custom_menu");
    if (theNumber >= 1) {
        //Find Custom Item and Append
        bbcode.ClearMenu(event.target);
        //Add custom Entried
        for (i=1; i <= theNumber; ++i) {
          var bcCustomLabel = unescape(this.getPrefBBCodeCompose().GetCharPref('bbcode.tags.custom.'+i+'.label'));
          var bcCustomImageURL = unescape(this.getPrefBBCodeCompose().GetCharPref('bbcode.tags.custom.'+i+'.url'));
          var menuitem = document.createElement("menuitem");
                menuitem.setAttribute("label",bcCustomLabel+" "); // Add a space at the end to prevent theme conflicts (like profiles named "home")
                menuitem.setAttribute("id","context-bbc-custom-"+i);
          menuitem.setAttribute("index",""+i+"");
                menuitem.setAttribute("oncommand","bbcode.bbcode('custom', 'bbcode', '"+bcCustomImageURL+"');");
                bbc_MainMenu.appendChild(menuitem);
        }
        //document.getElementById('context-bbc-custom').hidden = false;
    }else{
      // If changed to Zero again, erase and add Empty Field.
      //document.getElementById('context-bbc-custom').hidden = true;
      bbcode.ClearMenu(event.target);
      /*
      var menuitem = document.createElement("menuitem");
			menuitem.setAttribute("label",".....");
			menuitem.setAttribute("id","context-empty");
      menuitem.setAttribute("index","empty");
			bbc_MainMenu.appendChild(menuitem);
      */
    }
   return true;
  },

Diverse Debug-Versuche (theNumber und i ausgeben etc.) haben mich nicht auf die Spur gebracht. Irgendwann bin ich auf die Idee gekommen, das Menü zusätzlich vorher zurückzusetzen. Einmal langte scheinbar nicht (das Ergebnis war aber schon anders), aber doppelt hält's bekanntlich besser (bei 5 Custom Tags bisher keine Verdoppelungen mehr):

PopulateCustomImages: function(event){
    bbcode.ClearMenu(event.target);
    bbcode.ClearMenu(event.target);

(Rest der Funktion wie gehabt).

Hat dafür jemand eine Erklärung auf Lager? Ist das irgendein Timing-Problem?

Wie gesagt: Ich stehe da auch vor einem Rätsel. Wieviel "eigene Tags" hast du denn? und: Bei mir gab's auch einen doppelten Eintrag - der ging erst weg, nachdem ich die Zeile zweimal eingefügt habe.

Mir ist diese "Lösung" auch nicht ganz geheuer, aber solange es funktioniert, kann ich mit dem komischen Code leben (sieht ja keiner).

Mir fällt gerade etwas ein: Mit VBA kenne ich mich besser aus. Dort ist es so, wenn man eine "Collection" durchläuft und dabei alle Elemente löschen will, bleibt auch ca. die Hälfte der Elemente übrig. Lösung dabei ist: Die Sammlung von hinten nach vorn durchlaufen und anhand des Index löschen. Ich denke, dass könnte auch auf Javascript zutreffen:

ClearMenu: function(aParent) {
    var children = aParent.childNodes;
    for (i = 0; i < children.length; i++) {
        var index = children[i].getAttribute("index");
        if (index)
          aParent.removeChild(children[i]);
    }
  },

müsste also geändert werden. Bekommt das jemand von Euch hin? Also in VBA prinzipiell so:

For i=Anzahl der Elemente To 0 Step -1

Könnte

for (i = children.length; i <-1; i--) {

funktionieren?

Vielen Dank, Dr. Evil! Die ersten Versuche mit dem Code waren erfolgreich.

Ich hatte tatsächlich noch einen Dreher bzgl. "<" und sowieso Schwierigkeiten mit der Syntax.

ClearMenu: function(aParent) {
    var children = aParent.childNodes;
    for (i = children.length - 1; i > -1; i--) {
        var index = children[i].getAttribute("index");
        if (index)
          aParent.removeChild(children[i]);
    }
  },

so anpassen (s. For-Zeile) und die doppelten Zeilen von gestern aus "PopulateCustomImages" entfernen.

Aufgrund der Nachfrage unter http://www.firefox-browser.de/forum/viewtopi…p=404574#404574 hier noch einmal die hoffentlich richtig zusammengefassten Änderungen.

1.) Man entpacke die XPI in gewohnter Manier.
2.) Danach entpackt man die enthaltene JAR-Datei, die wiederum eine Datei bbcode.js enthält.

3.) In der bbcode.js sucht man diesen Part (Zeile 738) heraus:

ClearMenu: function(aParent) {
    var children = aParent.childNodes;
    for (i = 0; i < children.length; i++) {
        var index = children[i].getAttribute("index");
        if (index)
          aParent.removeChild(children[i]);
    }
  },

4.) Die unter 3. aufgeführten Zeilen durch folgenden Code ersetzen (eigentlich passt man nur die 740. Zeile mit "for ..." an):

ClearMenu: function(aParent) {
    var children = aParent.childNodes;
    for (i = children.length - 1; i > -1; i--) {
        var index = children[i].getAttribute("index");
        if (index)
          aParent.removeChild(children[i]);
    }
  },

5.) JAR erstellen, XPI erstellen und neu installieren.

Nachdem sich nun Mel Reyes (aka: mrtech) der Entwicklung von BBCode angenommen hat, geht es jetzt auch wieder voran mit neuen Versionen und BugFixes. Dieser alte Bug ist inzwischen auch in der neuen "Original"-Version behoben.

https://addons.mozilla.org/en-US/firefox/addon/128

Danke fürs Melden! Wenn nun noch

Zitat

the extra menu is definitely something that has annoyed me, where BBCodeXtra has it in the main menu, which is what I had used for a while. I'll look at providing an option to include it in the main context

(s. http://forums.mozillazine.org/viewtopic.php?p=3282100#3282100) umgesetzt wird, ist alles palletti. Bis dahin bleibe ich noch bei meiner veränderten Version.

Das wird sicher schnell nachgeliefert *g*

Waren die Entwickler von BBCode und BBCodeXtra früher nicht die gleichen Leute? Macht Mr Tech auch bei BBCodeXtra weiter?
Ich würde gerne mal eine Funktion von BBCode in BBCodeXtra einbauen, schaffe das aber selbst nicht (die Eigenen Tags) und die Email (*.it), die ich irgendwo als Kontakt gefunden habe, existiert nicht mehr.

BBCode wurde von Jed Brown entwickelt. Hinter BBCodeXtra steht Francesco Lodolo. Letzterer ist in den Credits von Jed Brown nicht genannt. So weit ich weiß, betreut mr.tech nur BBCode weiter. Die Mail-Adresse für den Entwickler hinter BBCodeXtra ist info [at] extenzilla [dot] org laut der entsprechenden Seite. Nachdem jetzt aber BBCode weiter entwickelt wird, ist BBCodeXtra ohnehin nicht mehr so relevant.

aha, ich hatte die gleiche Adresse mit *.it gefunden. Na, dann versuchs ich nochmal bei der anderen. Danke.