Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
W dniu 16.03.2019 o 11:28, Sylwester Zdanowski pisze:
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
Czy to aktualny kod lms z git z brancha master?
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Koncepcja jest w skrócie taka: - jeśli jest ustawiony weryfikator to on jedynie może zmienić deadline, - jeśli nie ma ustawionego weryfikatora każdy inny użytkownik może zmienić deadline, - w momencie gdy nie ma przypisanego weryfikatora powinieneś móc zmienić deadline ale tylko z przyszłości,
Witam, Inaczej, aktualna wersja rtmessageadd.php linia 74
if ($message['state'] != RT_RESOLVED && $message['deadline'] && ($deadline = datetime_to_timestamp($message['deadline']))) { if (!ConfigHelper::checkConfig('phpui.helpdesk_allow_all_users_modify_deadline') && $message['verifierid'] != Auth::GetCurrentUser())
($message['verifierid'] != Auth::GetCurrentUser() && $message['verifierid']!=0)
Patrząc na to jak próbowałem tego użyć, dodanie sprawdzenia czy przy ustawionym deadline verifierid nie jest czasem puste działało by lepiej.(domyślnie 0 wchodzi w verifierid)
Jeżeli ma być tak jak to cóż, można to wyłączyć konfiguracją.
W dniu 16.03.2019, sob o godzinie 16∶50 +0100, użytkownik Jarosław Kłopotek - INTERDUO napisał:
W dniu 16.03.2019 o 11:28, Sylwester Zdanowski pisze:
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
Czy to aktualny kod lms z git z brancha master?
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Koncepcja jest w skrócie taka:
- jeśli jest ustawiony weryfikator to on jedynie może zmienić
deadline,
- jeśli nie ma ustawionego weryfikatora każdy inny użytkownik może
zmienić deadline,
- w momencie gdy nie ma przypisanego weryfikatora powinieneś móc
zmienić deadline ale tylko z przyszłości,
Gdzie leży problem w: https://github.com/lmsgit/lms/blob/master/modules/rtmessageadd.php#L74-L80
"Patrząc na to jak próbowałem tego użyć, dodanie sprawdzenia czy przy ustawionym deadline verifierid nie jest czasem puste działało by lepiej.(domyślnie 0 wchodzi w verifierid)"
Jak byś sugerował?
W dniu 16.03.2019 o 20:03, Sylwester Zdanowski pisze:
Witam, Inaczej, aktualna wersja rtmessageadd.php linia 74
if ($message['state'] != RT_RESOLVED && $message['deadline'] && ($deadline = datetime_to_timestamp($message['deadline']))) { if (!ConfigHelper::checkConfig('phpui.helpdesk_allow_all_users_modify_deadline') && $message['verifierid'] != Auth::GetCurrentUser())
($message['verifierid'] != Auth::GetCurrentUser() && $message['verifierid']!=0)
Patrząc na to jak próbowałem tego użyć, dodanie sprawdzenia czy przy ustawionym deadline verifierid nie jest czasem puste działało by lepiej.(domyślnie 0 wchodzi w verifierid)
Jeżeli ma być tak jak to cóż, można to wyłączyć konfiguracją.
W dniu 16.03.2019, sob o godzinie 16∶50 +0100, użytkownik Jarosław Kłopotek - INTERDUO napisał:
W dniu 16.03.2019 o 11:28, Sylwester Zdanowski pisze:
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
Czy to aktualny kod lms z git z brancha master?
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Koncepcja jest w skrócie taka:
- jeśli jest ustawiony weryfikator to on jedynie może zmienić
deadline,
- jeśli nie ma ustawionego weryfikatora każdy inny użytkownik może
zmienić deadline,
- w momencie gdy nie ma przypisanego weryfikatora powinieneś móc
zmienić deadline ale tylko z przyszłości,
lms mailing list lms@lists.lms.org.pl http://lists.lms.org.pl/mailman/listinfo/lms
W dniu 17.03.2019, nie o godzinie 07∶51 +0100, użytkownik Jarosław Kłopotek - INTERDUO napisał:
Gdzie leży problem w: https://github.com/lmsgit/lms/blob/master/modules/rtmessageadd.php#L7 4-L80
"Patrząc na to jak próbowałem tego użyć, dodanie sprawdzenia czy przy ustawionym deadline verifierid nie jest czasem puste działało by lepiej.(domyślnie 0 wchodzi w verifierid)"
Jak byś sugerował?
Opisałem problem, podałem sugestie rozwiąznia tego co widzę. Uważasz że jest ok, nie chce Ci się czytać, sprawdzić? Nie mój problem. :)
W dniu 16.03.2019 o 20:03, Sylwester Zdanowski pisze:
Witam, Inaczej, aktualna wersja rtmessageadd.php linia 74
if ($message['state'] != RT_RESOLVED && $message['deadline'] && ($deadline = datetime_to_timestamp($message['deadline']))) { if (!ConfigHelper::checkConfig('phpui.helpdesk_allow_all_users_modify_ deadline') && $message['verifierid'] != Auth::GetCurrentUser())
($message['verifierid'] != Auth::GetCurrentUser() && $message['verifierid']!=0)
Patrząc na to jak próbowałem tego użyć, dodanie sprawdzenia czy przy ustawionym deadline verifierid nie jest czasem puste działało by lepiej.(domyślnie 0 wchodzi w verifierid)
Jeżeli ma być tak jak to cóż, można to wyłączyć konfiguracją.
W dniu 16.03.2019, sob o godzinie 16∶50 +0100, użytkownik Jarosław Kłopotek - INTERDUO napisał:
W dniu 16.03.2019 o 11:28, Sylwester Zdanowski pisze:
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
Czy to aktualny kod lms z git z brancha master?
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Koncepcja jest w skrócie taka:
- jeśli jest ustawiony weryfikator to on jedynie może zmienić
deadline,
- jeśli nie ma ustawionego weryfikatora każdy inny użytkownik
może zmienić deadline,
- w momencie gdy nie ma przypisanego weryfikatora powinieneś móc
zmienić deadline ale tylko z przyszłości,
lms mailing list lms@lists.lms.org.pl http://lists.lms.org.pl/mailman/listinfo/lms
W dniu 17.03.2019 o 08:50, Sylwester Zdanowski pisze:
W dniu 17.03.2019, nie o godzinie 07∶51 +0100, użytkownik Jarosław Kłopotek - INTERDUO napisał:
Gdzie leży problem w: https://github.com/lmsgit/lms/blob/master/modules/rtmessageadd.php#L7 4-L80
"Patrząc na to jak próbowałem tego użyć, dodanie sprawdzenia czy przy ustawionym deadline verifierid nie jest czasem puste działało by lepiej.(domyślnie 0 wchodzi w verifierid)"
Jak byś sugerował?
Opisałem problem, podałem sugestie rozwiąznia tego co widzę. Uważasz że jest ok, nie chce Ci się czytać, sprawdzić? Nie mój problem. :)
Nie wyciągaj mylnych wniosków. Czasy omawiania kodu na listach mailngowych skończyły się chyba już dawno. Czy nie prościej wrzucić tą drobną zmianę jako PR na githuba? Wtedy będę mógł ją sprawdzić na żywym organiźmie. Nasze gałęzie się różnią - u mnie ten problem nie występuje.
Pozdrawiam
W dniu 16.03.2019 11:28, Sylwester Zdanowski napisał(a):
Witam,
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Możesz sprawdzić czy bieżąca wersja kodu rozwiązuje problem?
Witam, Trafiłem na ciekawostkę.
Jeżeli ustawię termin ostateczny bez weryfikatora potem przy dodawaniu nowej wiadomości próbuję ustawić weryfikatora wyrzuca błąd "Jeżeli weryfikator jest ustawiony, jest on jedyna osobą która może zmienić termin ostateczny" Jednocześnie znika nagłówek strony "Nowa wiadomość". Weryfikatora można ustawić przez edycję zgłoszenia.
Teraz mając Termin ostateczny na 2019/03/31 11:35 i Weryfikatora na innego usera lms nie mogę dodać wiadomości bez wyświetlenia żadnego błędu.
zrobiłem print_r($error) Array ( [deadline] => Jeśli weryfikator jest ustawiony, jest on jedyną osobą, która może zmienić termin ostateczny! )
chociaż terminu nie da się w tym momęcie edytować...
W dniu 19.03.2019, wto o godzinie 08∶41 +0100, użytkownik Tomasz Chiliński napisał:
W dniu 16.03.2019 11:28, Sylwester Zdanowski napisał(a):
Witam,
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Możesz sprawdzić czy bieżąca wersja kodu rozwiązuje problem?
W dniu 19.03.2019 11:53, Sylwester Zdanowski napisał(a):
Witam,
Cześć,
Trafiłem na ciekawostkę.
Jeżeli ustawię termin ostateczny bez weryfikatora potem przy dodawaniu nowej wiadomości próbuję ustawić weryfikatora wyrzuca błąd "Jeżeli weryfikator jest ustawiony, jest on jedyna osobą która może zmienić termin ostateczny" Jednocześnie znika nagłówek strony "Nowa wiadomość". Weryfikatora można ustawić przez edycję zgłoszenia.
Teraz mając Termin ostateczny na 2019/03/31 11:35 i Weryfikatora na innego usera lms nie mogę dodać wiadomości bez wyświetlenia żadnego błędu.
zrobiłem print_r($error) Array ( [deadline] => Jeśli weryfikator jest ustawiony, jest on jedyną osobą, która może zmienić termin ostateczny! )
chociaż terminu nie da się w tym momęcie edytować...
Przeoczyłem jedną rzecz: https://github.com/lmsgit/lms/commit/a580125395b16cc9ab2f62eb45e1dd2a715367c... Nadal nie waliduje to zmiany weryfikatora i terminu ostatecznego dla wiadomości grupowej (brak walidacji), ale przynajmniej powinno działać dla pojedynczej wiadomości. Sprawdzisz teraz?
W dniu 19.03.2019, wto o godzinie 08∶41 +0100, użytkownik Tomasz Chiliński napisał:
W dniu 16.03.2019 11:28, Sylwester Zdanowski napisał(a):
Witam,
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Możesz sprawdzić czy bieżąca wersja kodu rozwiązuje problem?
lms mailing list lms@lists.lms.org.pl http://lists.lms.org.pl/mailman/listinfo/lms
W dniu 19.03.2019, wto o godzinie 12∶56 +0100, użytkownik Tomasz Chiliński napisał:
W dniu 19.03.2019 11:53, Sylwester Zdanowski napisał(a):
Witam,
Cześć,
Witam,
Trafiłem na ciekawostkę.
Jeżeli ustawię termin ostateczny bez weryfikatora potem przy dodawaniu nowej wiadomości próbuję ustawić weryfikatora wyrzuca błąd "Jeżeli weryfikator jest ustawiony, jest on jedyna osobą która może zmienić termin ostateczny" Jednocześnie znika nagłówek strony "Nowa wiadomość". Weryfikatora można ustawić przez edycję zgłoszenia.
Teraz mając Termin ostateczny na 2019/03/31 11:35 i Weryfikatora na innego usera lms nie mogę dodać wiadomości bez wyświetlenia żadnego błędu.
zrobiłem print_r($error) Array ( [deadline] => Jeśli weryfikator jest ustawiony, jest on jedyną osobą, która może zmienić termin ostateczny! )
chociaż terminu nie da się w tym momęcie edytować...
Przeoczyłem jedną rzecz: https://github.com/lmsgit/lms/commit/a580125395b16cc9ab2f62eb45e1dd2a 715367c0 Nadal nie waliduje to zmiany weryfikatora i terminu ostatecznego dla wiadomości grupowej (brak walidacji), ale przynajmniej powinno działać dla pojedynczej wiadomości. Sprawdzisz teraz?
W tej chwili nie widzę żadnego problemu.
W dniu 19.03.2019, wto o godzinie 08∶41 +0100, użytkownik Tomasz Chiliński napisał:
W dniu 16.03.2019 11:28, Sylwester Zdanowski napisał(a):
Witam,
Witam,
Być może nie widzę koncepcji autora.
Mam zgłoszenie z polem "Werfikator: żaden" Jak dodaje nową wiadomość dostaję "If verifier is set then he's the only person who can change deadline" (Jeżeli WERYFIKATOR JEST USTAWIONY....)
W rtmessageadd mamy w lini 74 ... && $message['verifierid'] != Auth::GetCurrentUser())
w praktyce bez ustawionego weryfikatora i bez phpui.helpdesk_allow_all_users_modify_deadline nie da się dodać wiadomości.
(Przy dodawaniu nowego zgłoszenia pole terminu ostatecznego automatycznie ustawia się na czas bieżący)
Pytanie czy powinien być dodatkowy warunek dla braku weryfikatora czy bez weryfikatora ma być nie możliwe dodanie wiadomości o ile ktoś nie ustawi konfiguracji? Jaka była koncepcja?
Możesz sprawdzić czy bieżąca wersja kodu rozwiązuje problem?
lms mailing list lms@lists.lms.org.pl http://lists.lms.org.pl/mailman/listinfo/lms
participants (3)
-
Jarosław Kłopotek - INTERDUO -
Sylwester Zdanowski -
Tomasz Chiliński