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

Terminarz - brak możliwości edycji zdarzenia #901

Closed
interduo opened this issue Sep 14, 2016 · 19 comments
Closed

Terminarz - brak możliwości edycji zdarzenia #901

interduo opened this issue Sep 14, 2016 · 19 comments

Comments

@interduo
Copy link
Collaborator

Złe działanie przycisku edycja z poziomu informacji o zdarzeniu. Jeśli jesteśmy lmsplus/?m=eventinfo&id=7, przycisk EDYTUJ nie posiada numeru zdarzenia (id) w linku.

@chilek
Copy link
Owner

chilek commented Sep 14, 2016

W której wersji?

@interduo
Copy link
Collaborator Author

aktualna z git teraz, branch master

@chilek
Copy link
Owner

chilek commented Sep 14, 2016

Sprawdziłem u siebie - problem nie występuje.

@interduo
Copy link
Collaborator Author

interduo commented Sep 14, 2016

Możliwe że to przez ten warning ze smarty (math x function)?

U mnie tak to wygląda:
terminarz-brakid

@interduo
Copy link
Collaborator Author

interduo commented Dec 14, 2016

Wyszło nowe smarty które rozwiązuje ten problem. Jesteś w stanie podbić smarty do najnowej wersji (3.1.31 z 23.11.2016)?

@chilek
Copy link
Owner

chilek commented Dec 14, 2016

Nie ma jeszcze smarty 3.1.31 w composerowym repo. Możesz sam próbować co jakiś czas modyfikując wiersz:
https://github.com/chilek/lms-plus/blob/master/composer.json#L26

@interduo
Copy link
Collaborator Author

interduo commented Dec 14, 2016

Update paczki ze smarty do -dev jednak też nie pomaga na ten problem.

Co ciekawe problem występuje u mnie na dwóch instancjach LMS na Debianie Jessie.

Jak to mogę zdebugować?

@interduo
Copy link
Collaborator Author

@chilek pokaż proszę jaką masz wersję pcre na demo2

Czy jesteś w stanie zastąpić funkcję {math}:
https://github.com/lmsgit/lms/blob/master/templates/default/event/eventinfo.html#L35
oraz
https://github.com/lmsgit/lms/blob/master/templates/default/event/eventinfo.html#L52

Przez inną funkcję lub stosując gołe obliczenia ze smarty?

@chilek
Copy link
Owner

chilek commented Dec 14, 2016

@interduo: nie prościej w środowisku produkcyjnym wyłączyć warningi php?

PCRE:
[I] dev-libs/libpcre
Available versions: (3) 8.38-r1 8.39
{bzip2 +cxx +jit libedit pcre16 pcre32 +readline +recursion-limit static-libs unicode zlib ABI_MIPS="n32 n64 o32" ABI_PPC="32 64" ABI_S390="32 64" ABI_X86="32 64 x32"}
Installed versions: 8.39(3)(12:10:04 PM 12/12/2016)(bzip2 cxx jit readline recursion-limit unicode zlib -libedit -pcre16 -pcre32 -static-libs ABI_MIPS="-n32 -n64 -o32" ABI_PPC="-32 -64" ABI_S390="-32 -64" ABI_X86="32 -64 -x32")
Homepage: http://www.pcre.org/
Description: Perl-compatible regular expression library

@interduo
Copy link
Collaborator Author

@chilek gdy nie wyświetla błędów nadal problem istnieje
Link pod przyciskiem edytuj nie ma wypełnionego ID a pola nie mają wypełnionych danych.

@chilek
Copy link
Owner

chilek commented Dec 14, 2016

@interduo: czy na demo2 problem też występuje? Jeśli tak to podaj kroki jakie wykonać?

@interduo
Copy link
Collaborator Author

Na demo2 problem nie występuje. Szukam więc różnic.

@interduo
Copy link
Collaborator Author

interduo commented Dec 14, 2016

Moje zgłoszenie na repo smarty smarty-php/smarty#288 zostało zamkniętę i teoretycznie w '3.1.31-dev' (od 11.09.2016) powinno to działać. Composer nie ma tej gałęzi, nie ma też najnowszej wersji.

Zastanawiam się czy to na pewno wina samego smarty.

@chilek U siebie na demo2 jaką masz wersję smarty?

Co z tym mogę zrobić?

@interduo
Copy link
Collaborator Author

interduo commented Dec 14, 2016

Zmiana w composer.json na dev-master też nie pomogła.
Na forach smartowych sugerują zastąpienie wolnej funkcji math poprzez vaniliowe obliczenia.

@interduo
Copy link
Collaborator Author

interduo commented Dec 16, 2016

Zrobiłem tak dodałem w kodzie:

<h1>EVENT.DATE {$event.date} EVENT.BEGINTIME {$event.begintime} EVENT.ENDTIME {$event.endtime}</h1>

Nie wyrzuca wartości zmiennych na ekran.

Jak tylko dodam na początku w kodzie {$event.date="0"} wartość pola "Czas rozpoczęcia" wyświetla się prawidłowo.

@interduo
Copy link
Collaborator Author

Smarty nie dostaje wartości z bazy z jakiegoś powodu, niezależnie od tego czy w bazie siedzą zera czy inne dane - coś jest nie tego z eventinfo.php/eventprint.html bo w innych modułach (np. eventedit,eventprint) jak najbardziej te dane są dostępne.

@chilek masz pomysł gdzie może leżeć problem?

@interduo
Copy link
Collaborator Author

interduo commented Dec 19, 2016

Znalazłem chwilę przejrzałem po kolei cały proces.
Błąd jest w zapytaniu SQL ale jeszcze nie doszedłem dlaczego się wykrzacza.

2016-12-16 13:35:28 CET [2222-1] lmsu@lmsdb BŁĄD:  kolumna events.closeddate nie istnieje przy znaku 279
2016-12-16 13:35:28 CET [2222-2] lmsu@lmsdb WYRAŻENIE:  SELECT events.id AS id, title, description, note, userid, events.creationdate,
					customerid, date, begintime, enddate, endtime, private, closed, events.type, UPPER(c.lastname) || ' ' || c.name AS customername,
				    vusers.name AS username, events.moddate, events.moduserid, events.closeddate, events.closeduserid, nodes.location AS location, c.city || ', ' || c.address AS customerlocation,
				    (SELECT name FROM vusers WHERE id=events.moduserid) AS modusername,
				    (SELECT name FROM vusers WHERE id=events.closeduserid) AS closedusername
				    FROM events
				    LEFT JOIN nodes ON (nodeid = nodes.id)
				    LEFT JOIN customerview c ON (c.id = customerid)
				    LEFT JOIN vusers ON (vusers.id = userid)
				    WHERE events.id = '8'

@interduo
Copy link
Collaborator Author

interduo commented Dec 19, 2016

@chilek na demo2 masz postgresa 9.5.5 a ja mam u siebie 9.4 i zapewne tutaj leży problem.

@interduo
Copy link
Collaborator Author

interduo commented Dec 20, 2016

Na Postgresql 9.6 nie występuje problem. Zrobiłem aktualizację sobie i zamykam temat.

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

2 participants