search

moodle-an-hochschulen / moodle-theme_boost_union

Theme Boost Union is an enhanced child theme of Boost which is intended, on the one hand, to make Boost simply more configurable and, on the other hand, to provide helpful additional features for the daily Moodle operation of admins, teachers and students.
GNU General Public License v3.0
66 stars 58 forks source link

Bug: Fehler beim Erzeugen der scss-Datein bei fehlendem Wert für blockregionoutsideleftwidth oder blockregionoutsiderightwidth #326

Closed HoffmannP closed 1 year ago

HoffmannP commented 1 year ago

Aus nicht mehr nachvollziehbaren Gründen waren in unserer Datenbank die Werte blockregionoutsideleftwidth und blockregionoutsiderightwidth nicht gesetzt. Daraufhin wurden die entsprechenden Variablen in scss nicht gesetzt und beim Kompilieren der scss-Datei kam es zu einem Fehler der leider still und leise im Äther verschwand. Ich schlage vor, wenn die Variablen nicht gesetzt sind (bzw leer), dann sollte der Default-Wert ausgegeben werden und nicht etwa die Variable im SCSS ausgelassen.

Betrifft https://github.com/moodle-an-hochschulen/moodle-theme_boost_union/blob/5326b632226b2c466319200b1762ce70a1d2237d/lib.php#L204-L206 und https://github.com/moodle-an-hochschulen/moodle-theme_boost_union/blob/5326b632226b2c466319200b1762ce70a1d2237d/lib.php#L207-L209

Grundsätzlich kann das Problem, dass SCSS-Variablen fehlen, aber auch an anderen Stellen in der gleichen Funktion auftauchen. Hier wäre eine Fehlermeldung sinnvoll, da selbst die Debugging-Ausgabe von SCSS nicht gut darauf hinweist, wo der Fehler aufgetaucht ist:

Undefined variable $blockregionoutsideleftwidth: line: 603, column: 13
Call Stack:
#0 mixin media-breakpoint-up (unknown file) on line 660
danowar2k commented 1 year ago

Seltsam, dass das Problem von zwei gleichzeitig bemerkt wird...Im Code sehe ich aber in der Richtung nichts auffälliges...

BerengarWLehr commented 1 year ago

Wie meinst du "nichts auffälliges"?

danowar2k commented 1 year ago

Ich meine damit, die settings.php sieht so aus, wie ich es erwarten würde, insbesondere die bei mir fehlerhaften/seltsam eingestellten Settings sehen nicht anders aus als ich sie erwarten würde...

danowar2k commented 1 year ago

Welche Version von Union war vorher drauf?

Unseren Aufzeichnungen zufolge bei uns 4.0-r8, danach 4.1-r7 jetzt. In Test, wo es keine Probleme gab, wurde wohl von v4.1-r2 auf v4.1-r7 aktualisiert. Seufz...Soviel zur Gleichheit der Systeme...

abias commented 1 year ago

Thank you for raising this issue!

I cannot imagine a case how the prerequisites for this bug could be triggered, but there was a gap in the code which sets the SCSS variables which I fixed in PR #329.

abias commented 1 year ago

This was fixed in #329