Spaw: target problem

[emergence]

ergänzung zu
-> http://www.contenido.de/forum/viewtopic ... ted+target

ich hab mir das jetzt angesehen und bin da auf etwas merkwürdiges gestossen...

der fehler läßt sich nur reproduzieren wenn mittels maus zwei wörter und ein leerzeichen ausgewählt wurden... mittels tastertur funktioniert es jedoch ganz wunderbar...

ähm auch nicht schlecht was der internet explorer da wieder zum besten gibt...

[emergence]

okay ich hab nen bugfix für den schwachsinn gefunden...
mit bitte um feedback...

in der contenido/external/wysiwyg/spaw/class/script.js.php
bei
function SPAW_hyperlink_click(editor, sender)

Code: Alles auswählen

	if(slink != null) {
      if (slink) {
        this[editor+'_rEdit'].document.execCommand('CreateLink',false,slink.Href); // Insert Url
        if (!hyp.target) { // try to get link
          var hyp = SPAW_getLink(editor);
        }
        if (hyp) {
          if (slink.Target != "") { // if target available set it
            hyp.target = slink.Target;
          } else { // remove it
            hyp.removeAttribute("target");
          }
        } else { // link capturing failed
          if (slink.Target != "") { // only warn if target was selected
            alert("Selected target could not be set. Select link once and try again.");
          }
        }
      }
    } else {
      this[editor+'_rEdit'].document.execCommand('UnLink',false);
    }

ersetzen durch

Code: Alles auswählen

	if(slink != null) {
      if (slink) {
        // v2.0 version modified version by horwath@opensa.org
        idstr = "556e697175657e537472696e67"; // new link creation ID, set HREF to this
        this[editor+'_rEdit'].document.execCommand('CreateLink',false,idstr);

        var oAnchors = this[editor+'_rEdit'].document.all.tags("A");

        if (oAnchors != null) {
          for (var i = oAnchors.length - 1; i >= 0; i--) {
            if (oAnchors[i] == idstr) {
              hyp = oAnchors[i];
              break;
            }
          }
        }

        hyp.href = slink.Href; // set selected hyperlink

        if (slink.Target != "") { // if target available set it
          hyp.target = slink.Target;
        } else { // remove it
          hyp.removeAttribute("target");
        }
      }
    } else {
      this[editor+'_rEdit'].document.execCommand('UnLink',false);
    }

anmerkung:
ein setzen des targets sollte jetzt immer möglich sein...
ich habs bis jetzt noch nicht geschafft die fehlermeldung zu reproduzieren...

[emergence]

eine kleine änderung hab ich bei dieser version noch ergänzt...

Code: Alles auswählen

	if(slink != null) {
      if (slink) {
        // v2.1 version modified version by horwath@opensa.org
        idstr = "556e697175657e537472696e67"; // new link creation ID, set HREF to this
        this[editor+'_rEdit'].document.execCommand('CreateLink',false,idstr);

        var oAnchors = this[editor+'_rEdit'].document.all.tags("A");

        if (oAnchors != null) {
          for (var i = oAnchors.length; i >= 0; i--) {
            if (oAnchors[i] == idstr) {
              hyp = oAnchors[i];

              hyp.href = slink.Href; // set selected hyperlink

              if (slink.Target != "") { // if target available set it
                hyp.target = slink.Target;
              } else { // remove it
                hyp.removeAttribute("target");
              }
            }
          }
        }

      }
    } else {
      this[editor+'_rEdit'].document.execCommand('UnLink',false);
    }

sobald man versucht sagen wir mal ein bild und text einen link gleichzeitig zugeben ,würde bei der ersten version nur der erste gefundene string wieder ersetzt...
sollte mit dieser änderung nicht auftreten...

[emergence]

hat das schon einer getestet ?

[timo]

bisher noch keine Zeit gehabt, mach ich aber die Tage mal

[Alex]

Hi!

Danke für das Bugfixing. Ich werde nächste Woche Dienstag mich dran setzen und melde mich dann, ob der Fehler noch besteht.


THANKS

Alex

[Alex]

Hi!

Sorry, das das so lange gedauert hat.

Aber es funktioniert. Ich habe es heute morgen eingespielt...

Vielen Dank

Alex...

[timo]

ja, schaut gut aus. Habe es eingebaut.