Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem z szablonami w LMS #547

Closed
kyob opened this issue Jan 16, 2016 · 25 comments
Closed

Problem z szablonami w LMS #547

kyob opened this issue Jan 16, 2016 · 25 comments

Comments

@kyob
Copy link
Contributor

kyob commented Jan 16, 2016

Przykładowo mam nowy katalog w templates/nowyszablon w nim jest:
welcome/welcome.html wprowadzone zmiany są widoczne, ale wszystko co jest includowane w pliku welcome.html np.

{include file="welcome/welcome_links.html"}

nie działa i ładuje pliki z domyślnego szablonu default

Mam ustawione: phpui.custom_templates_dir=nowyszablon

UPDATE: Szablony działają, jeśli dodam przedrostek nazwy szablonu

{include file="nowyszablon/welcome/welcome_links.html"}

Tak to ma działać?

@chilek
Copy link
Owner

chilek commented Jan 17, 2016

Dnia 17 stycznia 2016 00:01:26 CET, "Łukasz Kopiszka" notifications@github.com napisał(a):

Przykładowo mam nowy katalog w templates/nowyszablon w nim jest:
welcome/welcomehtml wprowadzone zmiany są widoczne, ale wszystko co
jest includowane w pliku welcomehtml np

{include file="welcome/welcome_linkshtml"}

nie działa i ładuje pliki z domyślnego szablonu default

Mam ustawione: phpuicustom_templates_dir=nowyszablon


Reply to this email directly or view it on GitHub:
#547

Jak wygląda struktura katalogów i plików w nowyszablon?

Pozdrawiam z Samsung Galaxy Note 4
Tomasz Chiliński, Chilan

@kyob
Copy link
Contributor Author

kyob commented Jan 17, 2016

cp -pr templates/default templates/nowyszablon

@chilek
Copy link
Owner

chilek commented Jan 17, 2016

Możesz spróbować wykonać test z lms z gałęzi smarty-3.1.29?
Przed chwilą puściłem pierwszy commit:
145bc4f
w tej nowo utworzonej gałęzi i jest spora szansa, że 3.1.29 rozwiąże opisywany przez Ciebie problem, a na pewno rozwiąże problem z rozpoznawaniem zmian w szablonach spowodowanym przez zastosowanie ze smarty 3.1.27 prefiltru, który był konieczny, a w 3.1.29 nie powinien już być potrzebny.
Po przejściu na tą gałąź trzeba zrobić:

  1. Zainstalować nową wersję smarty:
devel/smarty_install.sh
  1. Na wszelki wypadek:
rm -f templates_c/*
rm -f userpanel/templates_c/*

@kyob
Copy link
Contributor Author

kyob commented Jan 18, 2016

Teraz domyślny szablon nie ładuje podstron. Jakby to miało znaczenie testuje z PHP 5.5.9

@chilek
Copy link
Owner

chilek commented Jan 18, 2016

  1. Masz aktywne jakieś wtyczki? Jeśli tak to spróbuj tymczasowo wyłączyć.
  2. Wyłącz własny katalog szablonów i wtedy zobacz co wyświetli domyślny szablon.
  3. Nie masz żadnych błędów i/lub ostrzeżeń w przeglądarce www / logach serwera www?

@prezeskk
Copy link
Contributor

Potwierdzam ze tez pusta strona (znaczy sie jest tylko menu i naglowek bez srodka)
Najnowszy slackware 14.1 czysta testowa instalacja bez zadnych zmian

@chilek
Copy link
Owner

chilek commented Jan 18, 2016

Sprawdzę dziś na wersji demo.

@kyob
Copy link
Contributor Author

kyob commented Jan 18, 2016

  1. Brak wtyczek czyste repo.
  2. Domyślny szablon nie działa - tylko główna strona (lewe menu, gorny toolbar i inputy do szukania). Nie ładuje żadnych includowanych podstron.
  3. Brak błedów error_reporting = E_ALL

@prezeskk
Copy link
Contributor

To samo u mnie
Zaktualizowalem Slackware do wersji current
PHP: 5.6.15
Apache: 2.4.17
MariaDB: 10.0.22

@chilek
Copy link
Owner

chilek commented Jan 18, 2016

@kyob: a Ty jakiej dystrybucji używasz? ;-)

@kyob
Copy link
Contributor Author

kyob commented Jan 18, 2016

Testuje na:
Ubuntu 14.04.3 LTS :-)
Apache/2.4.12
PHP 5.5.9
MariaDB 10.1.10
Smarty 3.1.29

PS: Tak na marginesie Twoje zmiany działają ze smarty 3.1.27 tylko pluje błędami.

@chilek
Copy link
Owner

chilek commented Jan 18, 2016

A nie pisałem wiele razy, że Slackware i wszelkie pochodne Debiana to nie dystrybucje? :D

@prezeskk
Copy link
Contributor

Sprawdzam na 2gim systemie testowym
Debian 8.2
PHP: 5.6.14
Apache: 2.4.10
MySQL: 5.5.46

@prezeskk
Copy link
Contributor

Nigdy wczesniej problemow nie bylo ze Slackware
Debug smarty pokazuje poprawnie pobrane dane

@chilek
Copy link
Owner

chilek commented Jan 18, 2016

Hehe, przecież żartuję - to nie problem z dystrybucją - na demo też jest problem i siedzę właśnie nad tym. Prawdopodobnie kolejny błąd w Smarty ;-)

@chilek
Copy link
Owner

chilek commented Jan 18, 2016

Przygotowałem demonstrację problemu i wskazałem ją Uwe Tews ze Smarty:
smarty-php/smarty#123 (comment)

@nicraMarcin
Copy link
Contributor

Tomku, ten wątek w smarty jest zamknięty :/

@chilek
Copy link
Owner

chilek commented Jan 29, 2016

WIem o tym - od dawna jest zamknięty, ale jak Uwe ma wolną chwilę to do niego zagląda.

@kyob kyob mentioned this issue Feb 12, 2016
@chilek
Copy link
Owner

chilek commented Feb 12, 2016

Na wszelki wypadek utworzyłem nowe zgłoszenie:
smarty-php/smarty#183

@chilek
Copy link
Owner

chilek commented Feb 14, 2016

Wygląda na to, że problem rozwiązany. Można testować zmiany w gałęzi smarty-3.1.29 (na demo działa). Jutro zaciągnę zmiany z tej gałęzi do master.

@prezeskk
Copy link
Contributor

Potwierdzam dzialanie ;]
Co do czasow generowania musze porobic troche testow ale na pierwszy rzut oka wyglada na to ze dziala szybciej

@kyob
Copy link
Contributor Author

kyob commented Feb 15, 2016

PHP Fatal error: Class 'Smarty_Internal_Undefined' not found in /var/www/lms/lib/Smarty/sysplugins/smarty_internal_extension_handler.php on line 154

sam sobie odpowiem: composer update :)

@pjona
Copy link
Contributor

pjona commented Feb 19, 2016

My też potwierdzamy działanie Smarty 3.1.29 z naszymi pluginami, po drobnych poprawkach.

@chilek
Copy link
Owner

chilek commented Feb 19, 2016

Tak - w pluginach trzeba będzie usunąć wszystkie:

{extends file="nazwa_pliku_z_szablonem"}

w przypadku, gdy nazwa_pliku_z_szablonem jest taka sama jak sam plik w którym to występuje.

@pjona
Copy link
Contributor

pjona commented Feb 19, 2016

Dokładnie.

@chilek chilek closed this as completed Mar 2, 2016
chilek added a commit that referenced this issue Apr 24, 2018
chilek added a commit that referenced this issue Apr 24, 2018
… assign and unassign (LMS+ #547) - changelog entry added
interduo pushed a commit to interduo/lms that referenced this issue Aug 16, 2018
… assign and unassign (LMS+ chilek#547) - changelog entry added
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants