[JS] Probleem met CKEditor

[Tim_R]

Ik heb een probleempje met de CKEditor, bij het bouwen van een nieuw CMS.

Het zit als volgt: de gebruiker voert een tekst in in de CKEditor en klik op de button 'Opslaan'. Een javascript-functie wordt aangeroepen:

JavaScript:

function submit(tabel,id){
              var naam  = 'tekst' + id;
              var tekst = 'CKEDITOR.instances.' + naam + 'getData(naam);';
              var tekst2 = tekst;
              $.ajax({
                url: 'models/wijzig.php?submit=true',
                type: 'POST',
                data: 'id=' + id + '&tekst=' + tekst2 + '&tabel=' + tabel,
                
                success: function(result) {
                    var meldingdiv = '.melding' + id;
                    var meldingstekstdiv = '.meldingstekst' + id;
                    $(meldingdiv).slideUp(function(){
                        $(meldingdiv).removeClass('mislukt').slideDown();
                        $(meldingdiv).removeClass('geslaagd').slideDown();
                        $(meldingdiv).removeClass('info').slideDown();
                        $(meldingdiv).removeClass('waarschuwing').slideDown();
                        $(meldingstekstdiv).empty();
                        $(meldingstekstdiv).append('Succesvol opgeslagen!');
                        $(meldingdiv).addClass('geslaagd').slideDown();
                    });
                }
              });
            return false;
        }

Er staan meedere editors op 1 pagina, vandaar dat de CKEditors allemaal een apart id dienen te krijgen ('tekst'+ id van het item).
Wat dus lukt op regel 3 is het genereren van de tekst die 'uitgevoerd' moet worden maar het lukt me op geen enkele manier om het ook daadwerkelijk uit te laten voeren!

Iemand een idee?

[MueR]

Je stuurt nu gewoon een string mee. Je zult op een of andere manier de browser daadwerkelijk de referentie moeten laten evalueren als javascript.

[Tim_R]

MueR schreef op maandag 08 februari 2010 @ 21:49:
Je stuurt nu gewoon een string mee. Je zult op een of andere manier de browser daadwerkelijk de referentie moeten laten evalueren als javascript.

Klopt! Maar aangezien ik meestal alleen werk met jQuery en niet echt into die-hard-js ben hoopte ik dat iemand een gouden tip zou hebben over hoe..

[Tim_R]

Ja! 'Evalueren' was het missende sleutelwoord, ik vond http://www.w3schools.com/jsref/jsref_eval.asp en dat was precies wat ik nodig had. Bedankt!

For the record nog de aangepaste code:

JavaScript:

function submit(tabel,id){
              var naam  = 'tekst' + id;
              var tekst = 'CKEDITOR.instances.' + naam + '.getData();';
              var tekst2 = eval(tekst);
              $.ajax({
                url: 'models/wijzig.php?submit=true',
                type: 'POST',
                data: 'id=' + id + '&tekst=' + tekst2 + '&tabel=' + tabel,
                
                success: function(result) {
                    var meldingdiv = '.melding' + id;
                    var meldingstekstdiv = '.meldingstekst' + id;
                    $(meldingdiv).slideUp(function(){
                        $(meldingdiv).removeClass('mislukt').slideDown();
                        $(meldingdiv).removeClass('geslaagd').slideDown();
                        $(meldingdiv).removeClass('info').slideDown();
                        $(meldingdiv).removeClass('waarschuwing').slideDown();
                        $(meldingstekstdiv).empty();
                        $(meldingstekstdiv).append('Succesvol opgeslagen!');
                        $(meldingdiv).addClass('geslaagd').slideDown();
                    });
                }
              });
            return false;
        }

[pieturp]

Bah! eval === evil!

Doe alsjeblieft het volgende:

JavaScript:

var tekst = CKEDITOR.instances[naam].getData();

Overigens, jQuery ondersteund het chainen van methods

[MueR]

^^ die is beter inderdaad

Ik ben niet echt helder aan het denken vandaag

[Tim_R]

Okee, bedankt! Die werkt ook en dat lijkt me inderdaad netter dan eval(). De gelijkenis met 'evil' zal wel niet voor niets zijn dan..

Edit: Dat chainen klopt, dat heb ik inmiddels ook gedaan. Tijdens het ontwikkelen heb ik voor de overzichtelijkheid tijdens het debuggen even alles op aparte regels geplaatst. Thanks voor de tip though!

[ Voor 43% gewijzigd door Tim_R op 09-02-2010 22:29 ]

[pieturp]

NP

Overigens kun je ook prima leesbaar chainen, als je maar netjes inspringt:

JavaScript:

$(selector)
   .method(arg)
   .method(function(){
      //
   })
   .nogEenMethod(arg);

Vergeet alleen de ; en . niet een consequente plek te geven. Ik zet de punt vooraan, maar dat hoeft niet natuurlijk