fff-web: move includes to non-HTTPed folder and modularize WebUI #3

adschm · 2020-12-12T17:33:38+01:00

adschm commented

2020-12-12 17:33:38 +01:00

Dies verschiebt die includes aus dem Ordner, der per HTTP geservt wird, sodass man nicht mehr darauf zugreifen kann.
Eine veränderte Version von Christians Patch zum Modularisieren des WebUI.

1. Dies verschiebt die includes aus dem Ordner, der per HTTP geservt wird, sodass man nicht mehr darauf zugreifen kann. 2. Eine veränderte Version von Christians Patch zum Modularisieren des WebUI.

adschm self-assigned this 2020-12-12 17:33:38 +01:00

adschm added 2 commits 2020-12-12 17:33:38 +01:00

eaa810267d fff-web: remove included files from HTTP-accessible folder

Certain files are only included, but are not meant to be accessed
via HTTP directly. Move those to a dedicated directory that is not
served via HTTP.

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

0f898b9ab8 fff-web: Make ssl menu modular

To add new menu items in other packages we need a modular construction:
- Simpel Babelweb
- Layer 3 configuration
and so on

Every menu item needs a file in /www/menu/ssl/ with the content

  link,name

  - link is the name of the html file without extension (.html)
  - name is the text of the link in the webui

While at it, rename "Password" menu item to German "Passwort".

Signed-off-by: Christian Dresel <freifunk@dresel.systems>
[bump PKG_RELEASE, use /www/menu, use script_file for link,
improve use of class_active, commit message adjustments,
improve variable names, keep HTML umlaut, keep Logout]
Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>

fbl approved these changes 2020-12-12 18:39:15 +01:00

fbl left a comment

Warum habt ihr denn die nav_entry Funktion aufgelöst?
Ansonsten sieht das für mich so weit gut aus.

Warum habt ihr denn die nav_entry Funktion aufgelöst? Ansonsten sieht das für mich so weit gut aus.

adschm commented

2020-12-12 18:40:38 +01:00

Warum habt ihr denn die nav_entry Funktion aufgelöst?

Weil es jetzt nur noch einen Aufruf gäbe ...

> Warum habt ihr denn die nav_entry Funktion aufgelöst? Weil es jetzt nur noch einen Aufruf gäbe ...

ChristianD approved these changes 2020-12-12 20:29:18 +01:00

src/packages/fff/fff-web/files/www/include/header

						
				@ -0,0 +52,4 @@

							menu_link="/cgi-bin/${menu_file}.html";

							class_active=""

							[ "$menu_link" = "$REQUEST_URI" ] && class_active=' class="active"'

							echo -ne "\t<li$class_active><a href=\"${menu_link}\">$menu_name</a></li>\n\t\t";

ChristianD commented

2020-12-12 18:20:04 +01:00

sicher das beim href das cgi-bin mit muss? Ich meine mich zu erinnern das "nur" der Dateiname richtig war, demnach statt {menu_link} wäre {menu_file}.html richtig

adschm commented

2020-12-12 20:30:53 +01:00

Es ist entweder
settings.html
oder
/cgi-bin/settings.html

Beides funktioniert, erstes ist relativ und letzteres absolut.

Ich habe letzteres beibehalten, weil es vorher so war und es keinen Grund zum Ändern gibt.

Es ist entweder settings.html oder /cgi-bin/settings.html Beides funktioniert, erstes ist relativ und letzteres absolut. Ich habe letzteres beibehalten, weil es vorher so war und es keinen Grund zum Ändern gibt.

ChristianD commented

2020-12-12 20:31:28 +01:00

ah ok wenn beides funktioniert kann ich mit leben, dann alles ok und weiter machen :)

fbl marked this conversation as resolved