Neues Plugin: PRISM (prismjs.com) SyntaxHighlighter for Joomla!

Zitat von 89hf4edc

Gebt mir gern Feedback.

Selber Schuld

Mich würde stören, dass CSS und JS auf jeder Seite geladen wird, egal, ob benötigt oder nicht. Vielleicht besser in onContentPrepare(...) (statt __construct(...)) mit paar vorgeschalteten Prüfungen wie $context, view, eh schon alles geladen (bspw. via static Variable festgehalten) und ein Quicktest auf Vorkommen von eventuell Code-Blöcken. Soweit ich sehe, haben die immer ein <pre und immer ein <code>

So was in der Art:

if (
 strpos($article->text, '<pre  ') === false
 || strpos($article->text, '<code>') === false
){
 // Abbruch. Sicher nix zu tun.
 return true;
}

J! 3.7 Alpha liefert Warnings aus:

Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 620


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 621


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 622


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 623


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 693


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 694


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 695


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 696


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 697


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 620


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 621


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 622


Warning:  Cannot use a scalar value as an array in C:\xampp\htdocs\3-7\libraries\cms\html\html.php on line 623

Ich seh die Warning zwar nicht (mit 3.7.0 Staging von gestern und der neuen Plugin-Version von heute), aber in den Methoden script und stylesheet hat sich Grundlegendes geändert. Das true (für lade framework) bspw. ist so zukünftig nicht mehr gewünscht, sondern man übergibt ein Array mit ('framework' => true, usw.)

Jetzt fällt auf, dass die Methode script hier eine Prüfung drinnen hat für B\C

public static function script($file, $options = array(), $attribs = array()){  // B/C before 3.7.0  if (!is_array($options) && !is_array($attribs))  {   JLog::add('The script method signature used has changed, use (file, options, attributes) instead.', JLog::WARNING, 'deprecated');

(https://github.com/joomla/joomla-…l/html.php#L676)

während die Methode stylesheet $options nicht auf is_array prüft (nur $attribs), was dann wohl zum Fehler führt.

public static function stylesheet($file, $options = array(), $attribs = array())
{
 // B/C before 3.7.0
 if (!is_array($attribs))
 {
  JLog::add('The stylesheet method signature used has changed, use (file, options, attributes) instead.', JLog::WARNING, 'deprecated');

(https://github.com/joomla/joomla-…l/html.php#L604)

Also, wohl ein Bug anzunehmen. Da ich leider nicht reproduzieren kann, kann ich auch nicht einreichen.

Wo denn einfügen? In welche Methode? Welche PHP-Version? XAMPP oder nicht?

(Ich verwende nicht das Alpha-Release sondern das Staging-ZIP (Download-Button), weil da ja schon schrittweise Bugs gefixt sind.)

Ursache ist, dass du die Zeile zu früh setzt. Weder im Constructor (war ja bei deienr Pluginversion von gestern noch der Fall), noch am Beginn der Datei, noch in onAfterInitialise(), noch in ...

Deshalb meine Frage, wo du exakt eingesetzt hast. Es gibt JHtml-Zeilen (auch andere), mit denen du das ganze System in Teilen fehlfunktionierend machen kannst, weil die aufgerufenen Methoden irgendwo auf ihrem Weg durch die Eingeweide und weitere Methoden zu früh z.B. ein $app->getTemplate() ausführen ("beliebtestes" Beispiel).

Ein
JHtml::_('jquery.framework');
JHtml::_('bootstrap.framework');
in onAfterInitialise() ist z.B. so ein Killer (allerdings ohne Warning, aber mit erheblichem, folgendem Rätselverhalten des Layout-Override-Systems).

Gehört frühestens in onAfterRoute().

Kurz: Programmierfehler. Kein Bug

Zitat von 89hf4edc

aus der Doku zu lesen, was man wann wie am Besten aufruft

Anekdötchen zum Feierabend:
Das steht wenn, dann nur zufällig so explizit in irgendeiner Doku. Ich werds nur deshalb nicht mehr vergessen, weil eine Zeit lang fast jedes meiner aufwendigeren Systemplugins einen solchen "schweigenden" Fehler drin hatte Erst nachdem ich die alle (weinend) umprogrammiert hatte, habe ich die Doku bzgl. Reihenfolge von Plugin-Typen/-Gruppen und Plugin-Events überhaupt richtig verstanden. Das meiste durch Spicken im Core und in Plugins von Leuten, auf die man sich (hoffentlich) verlassen kann, aus Forenproblemen und auf GitHub, wo's auch mal Thema war. Es gab/gibt auch ein Template-Framework, was diesbzgl. einen gravierenden Fehler drin hatte/hat. Da war/ist die plumpe Antwort vom Support "Fehler berichten" zu deaktivieren

Zitat von 89hf4edc

dass ich es gescheit mache

Hast ja jetzt eh schon korrigiert in deinem Plugin.