search

chilek / lms

Lan Management System (LMS) public GIT repo
http://lms.org.pl
126 stars 134 forks source link

enhancement: added SIDUSIS Maps link type, added variables: 'sidusis.web_url', 'sidusis.map_zoom_range' (default: 20) #2401

Closed interduo closed 9 months ago

interduo commented 10 months ago

…l' (without default value), 'sidusis.map_zoom_range' (with default value of '20')

PR dodaje obsługę link do mapy SIDUSIS/internet.gov.pl: image

Dodane zmienne:

sidusis.map_zoom_range

Typ: Wartość liczbowa.

Wartość domyślna: 20

Opis: Zmienna określa wskaźnik powiększenia załadowanej mapy po kliknięciu w link.

Dostępność: od 28.0

sidusis.web_url

Typ: Wartość teksowa.

Wartość domyślna: https://internet.gov.pl/map/?center=

Opis: Zmienna określa adres URL webserwisu SIDUSIS.

Dostępność: od 28.0

awbnet commented 10 months ago

Przyda się nam w biurze na pewno. Czekamy na review i merge.

interduo commented 10 months ago

  1. @chilek sugeruję zmianę parametru "default" na "externalmap" https://github.com/chilek/lms/blob/master/lib/SmartyPlugins/LMSSmartyPlugins.php#L1190 - zauważ, że niżej jest już "default" (innymi słowy brzydko jest stosować w wariantach parametrów słowa występujące w składni języka)

  2. Czemu usunąłeś funkcję pozwalającą nadpisać domyślną etykietkę lub podpowiedź w komponencie {show_on_map_button}?

  3. We wrzuconym kodzie przez Ciebie - parametr external w funkcji {show_on_map_button} nie działa - w moim PR działał.