Vragen over Moodle

Hans, we dwalen in deze discussie wel erg af van de Safari vraag: off topic

..ik heb wat je suggereerde opgezocht. Dit lijkt de plaats waar de switch tussen de htmlarea en de textarea wordt gemaakt.
(De check of de html-editor gebruikt kan worden is dan al gedaan.)

Hier zou je dus ook de plugin-ruimte/keuze-switch verwachten voor TinyMCE, maar ik zie hem niet, jammer, zou veel werk schelen. (ik neem als leek aan dat die technische check voor TinyMCE straks hetzelfde is en ook de gebruikers-keuze in het profiel identiek zal verlopen: in het profiel van de gebruiker zou je de keuze tussen beide editors als derde optie verwachten, zo ook in de algemene instellingen van de admin-pagina's.) 

Dat lijkt me echt werk voor de echte ontwikkelaars, die wellicht een plaats weten waar het nog beter kan worden ingevoegd. Op dit moment zie ik geen code, vriendelijk op één plaats, wel zie ik erg veel code herhaling:

in jouw lijn redenerend zou ik een objectklasse verwachten voor mod-form.php dat aangeroepen wordt door de modules, in werkelijkheid zie ik steeds dezelfde regels code herhaald worden in alle mod-forms (en bij een CVS update collectief gecorrigeerd worden, beseffend dat daar wel een programmeur bezig geweest moet zijn om op al die 20 plaatsen die code aan te passen.)

Er kan natuurlijk altijd een andere reden voor zijn, bijvoorbeeld performance, omdat object oriented in het verleden zelden tot de snelste code leidde.. 

Dit is wat ik vond in /lib/form/htmleditor.php van 1.9.2:

class MoodleQuickForm_htmleditor extends MoodleQuickForm_textarea{
    var $_type;
    var $_canUseHtmlEditor;
    var $_options=array('canUseHtmlEditor'=>'detect','rows'=>10, 'cols'=>45, 'width'=>0,'height'=>0, 'course'=>0);
    function MoodleQuickForm_htmleditor($elementName=null, $elementLabel=null, $options=array(), $attributes=null){
        parent::MoodleQuickForm_textarea($elementName, $elementLabel, $attributes);
        // set the options, do not bother setting bogus ones
        if (is_array($options)) {
            foreach ($options as $name => $value) {
                if (isset($this->_options[$name])) {
                    if (is_array($value) && is_array($this->_options[$name])) {
                        $this->_options[$name] = @array_merge($this->_options[$name], $value);
                    } else {
                        $this->_options[$name] = $value;
                    }
                }
            }
        }
        if ($this->_options['canUseHtmlEditor']=='detect'){
            $this->_options['canUseHtmlEditor']=can_use_html_editor();
        }
        if ($this->_options['canUseHtmlEditor']){
            $this->_type='htmleditor';
            //$this->_elementTemplateType='wide';
        }else{
            $this->_type='textarea';
        }
        $this->_canUseHtmlEditor = $this->_options['canUseHtmlEditor'];
    }
...