Report any technical problems you discover and discuss solutions.

New base template, default engine = flash

r.j.
Onderwerp Auteur
Offline
Junior lid

Meer

10 jaren 1 maand geleden - 10 jaren 1 maand geleden #3786 door r.j.

New base template, default engine = flash werd gestart door r.j.

When adding a new template (as in Nottingham variant), method found here , i find that every new LO template added gets the flash default engine..

I guess this is due to line 69 a.f. of /website_code/php/templates/new_template.php (see below).
Is there a reason to only make javascript default with templates named Nottingham and not every template of the type xerte?

/

Code:

*
 * create the new template record in the database
 */
$extraflags = "";
if ($row_template_type['template_framework'] == 'xerte')
{
    if ($row_template_type['template_name'] == 'Nottingham')
    {
        $extraflags = "engine=javascript";
    }
    else
    {
        $extraflags = "engine=flash";
    }
}

Laatst bewerkt 10 jaren 1 maand geleden door r.j..

Graag Inloggen of een account aanmaken deelnemen aan het gesprek.

JohnSmith
Offline
Moderator

Meer

10 jaren 1 maand geleden #3787 door JohnSmith

Beantwoord door JohnSmith in topic New base template, default engine = flash

Hi,

Those instructions are 2 1/2 years old and perhaps requiring some updating. The reason for extraflags = flash for all the non-Nottingham templates is because of legacy mediaInteractions and multipersp templates ( github.com/thexerteproject/xerteonlineto...8536070692bd6df9b233 ) which descend from Xerte but do not play in the new javascript/HTML5 player.

Perhaps it would be better to check for these rather than 'Nottingham' and flip the logic:

Code:

*
 * create the new template record in the database
 */
$extraflags = "";
if ($row_template_type['template_framework'] == 'xerte')
{
    if ( ($row_template_type['template_name'] == 'multipersp') ||
        ($row_template_type['template_name'] == 'mediaInteractions') ||
        ($row_template_type['template_name'] == 'Rss') )
    {
        $extraflags = "engine=flash";
    }
    else
    {
        $extraflags = "engine=javascript";
    }
}

De volgende gebruiker (s) zei dank u: r.j.

Graag Inloggen of een account aanmaken deelnemen aan het gesprek.

r.j.
Onderwerp Auteur
Offline
Junior lid

Meer

10 jaren 1 maand geleden #3788 door r.j.

Beantwoord door r.j. in topic New base template, default engine = flash

Far better, thanks!

Did you already commit this on github?

Graag Inloggen of een account aanmaken deelnemen aan het gesprek.

JohnSmith
Offline
Moderator

Meer

10 jaren 1 maand geleden #3789 door JohnSmith

Beantwoord door JohnSmith in topic New base template, default engine = flash

Nope just came up with it but I can do if you are happy that it is better?

Graag Inloggen of een account aanmaken deelnemen aan het gesprek.

JohnSmith
Offline
Moderator

Meer

10 jaren 1 maand geleden #3790 door JohnSmith

Beantwoord door JohnSmith in topic New base template, default engine = flash

That's it committed to Github.

Graag Inloggen of een account aanmaken deelnemen aan het gesprek.

Moderators: ronm, julten, JohnSmith

Tijd voor maken pagina: 0.128 seconden

Gemaakt door Kunena

New base template, default engine = flash

Main Pages

Quick Links

Support

External resources