Report any technical problems you discover and discuss solutions.

New base template, default engine = flash

r.j.
Auteur du sujet
Hors Ligne
Membre junior

Plus d'informations

il y a 10 ans 1 mois - il y a 10 ans 1 mois #3786 par r.j.

New base template, default engine = flash a été créé par 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";
    }
}

Dernière édition: il y a 10 ans 1 mois par r.j..

Connexion ou Créer un compte pour participer à la conversation.

JohnSmith
Hors Ligne
Modérateur

Plus d'informations

il y a 10 ans 1 mois #3787 par JohnSmith

Réponse de JohnSmith sur le sujet 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";
    }
}

Les utilisateur(s) suivant ont remercié: r.j.

Connexion ou Créer un compte pour participer à la conversation.

r.j.
Auteur du sujet
Hors Ligne
Membre junior

Plus d'informations

il y a 10 ans 1 mois #3788 par r.j.

Réponse de r.j. sur le sujet New base template, default engine = flash

Far better, thanks!

Did you already commit this on github?

Connexion ou Créer un compte pour participer à la conversation.

JohnSmith
Hors Ligne
Modérateur

Plus d'informations

il y a 10 ans 1 mois #3789 par JohnSmith

Réponse de JohnSmith sur le sujet New base template, default engine = flash

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

Connexion ou Créer un compte pour participer à la conversation.

JohnSmith
Hors Ligne
Modérateur

Plus d'informations

il y a 10 ans 1 mois #3790 par JohnSmith

Réponse de JohnSmith sur le sujet New base template, default engine = flash

That's it committed to Github.

Connexion ou Créer un compte pour participer à la conversation.

Modérateurs: ronm, julten, JohnSmith

Temps de génération de la page : 0.146 secondes

Propulsé par Kunena

New base template, default engine = flash

Main Pages

Quick Links

Support

External resources