compactor zum komprimieren der js-Files

[noRiddle (revilonetz)]

Hallo Community, hallo modified-Team.
Kann es sein, daß der compactor Doppel-Slash Auskommentierungen in Javascript-Files nicht filtert ?
Bekomme reihenweise Fehler wenn ich mit // auskommentiere, nicht jedoch wenn ich /*...*/ auskommentiere.
Das führt schnell mal zu einer endlosen Suche nach dem Fehler, da die Fehler-Aussagen in der Konsole teilweise sehr allgemein sind und im komprimierten File schwerlich eine Zeilennummer angeben werden kann.
Aussagen wie
"TypeError: (intermediate value)(...) is not a function" sind nicht besonders informativ.

Oder reicht es 'strip_php_comments' auf true zu setzen ?
(Ich weiß, könnte ich testen, habe lediglich Stress momentan. Test werde ich nachholen und den Thread up-date-n.)

Weiß jemand Genaueres ?

Gruß,
noRiddle

*NACHTRAG*
Dabei fällt mir gerade auf:
Wenn es mit dem "'strip_php_comments' auf true zu setzen" klappen würde, was passiert wenn man im Code so etwas hat (Beispiel colorbox.js):

Code: Javascript  [Auswählen]

{n.src="//about:blank"}

Linkback: https://www.modified-shop.org/forum/index.php?topic=36910.0

[noRiddle (revilonetz)]

:push:

[p3e]

Irgendwo hier war das mit dem Compactor und Doppelslash schon Thema. Wenn nicht im Forum dann im Bug-Tracker. Ich glaube, dass es die Überlegung gab, auf einen anderen Compactor zu wechseln, da der aktuelle seit längerem nicht mehr weiter entwickelt wird.

[noRiddle (revilonetz)]

Ah, danke dir p3e, du hast Recht, gibt's im Bug-Tracker, ich sollte da öfter reinschauen.

Gruß,
noRiddle

[web28]

Du kannst das ja mal testen und in der Compactor Klasse diese Zeile aktivieren

Code: PHP  [Auswählen]

$code = preg_replace('/[^:]\/\/.*/', '', $code);

Code: PHP  [Auswählen]

private function _simpleCodeCompress($code)
                {
      // Remove multiline comment
                        $code = preg_replace('/\/\*(?!-)[\x00-\xff]*?\*\//', '', $code);
      // Remove single line comment
      // $code = preg_replace('/[^:]\/\/.*/', '', $code);
                        $code = preg_replace('/\\/\\/[^\\n\\r]*[\\n\\r]/', '', $code);
                        $code = preg_replace('/\\/\\*[^*]*\\*+([^\\/][^*]*\\*+)*\\//', '', $code);
      // Remove extra spaces
                        $code = preg_replace('/\s+/', ' ', $code);
      // Remove spaces that can be removed
                        return preg_replace('/\s?([\{\};\=\(\)\/\+\*-])\s?/', "\\1", $code);
                }