Hej,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
W dniu 2016-01-05 o 17:28, Tomasz Chiliński pisze:
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L92
jest:
else
winno być:
if ($type == MSG_MAIL)
ponieważ:
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L145 dla wszystkich innych wiadomości innych niż SMS nie stworzy listy odbiorców wiadomości dla np. MSG_WWW, bo no klient nie ma podanego emaila. To samo dla MSG_USERPANEL.
Reasumując, nie podasz w karcie klienta maila to nic do niego nie wyślesz.
W dniu 06.01.2016 11:31, D.Wesołowski napisał(a):
W dniu 2016-01-05 o 17:28, Tomasz Chiliński pisze:
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L92
jest:
else
winno być:
if ($type == MSG_MAIL)
ponieważ:
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L145 dla wszystkich innych wiadomości innych niż SMS nie stworzy listy odbiorców wiadomości dla np. MSG_WWW, bo no klient nie ma podanego emaila. To samo dla MSG_USERPANEL.
Reasumując, nie podasz w karcie klienta maila to nic do niego nie wyślesz.
Słusznie.
https://github.com/lmsgit/lms/commit/768329d338e36999d2ca195270f214b87979c81...
Przy okazji inna poprawka eliminująca bezsensowny kod - musiał ktoś (lub ja) późną nocą to przepisywać ;-)
Btw. wyszło wczoraj 1.11.19.2, a powyższa poprawka weszła do master i będzie w 1.11.19.3.
W dniu 2016-01-06 o 11:31, D.Wesołowski pisze:
W dniu 2016-01-05 o 17:28, Tomasz Chiliński pisze:
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L92
jest:
else
winno być:
if ($type == MSG_MAIL)
ponieważ:
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L145 dla wszystkich innych wiadomości innych niż SMS nie stworzy listy odbiorców wiadomości dla np. MSG_WWW, bo no klient nie ma podanego emaila. To samo dla MSG_USERPANEL.
Reasumując, nie podasz w karcie klienta maila to nic do niego nie wyślesz.
aby nie generować wiadomości do klientów nie posiadających komputerów (nam się tacy zdarzają ;) za linią https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L146 dodałbym
. ($type == MSG_WWW ? 'JOIN (SELECT ownerid FROM `nodes` GROUP BY ownerid) cn ON (cn.ownerid = c.id) '
W dniu 06.01.2016 12:55, D.Wesołowski napisał(a):
W dniu 2016-01-06 o 11:31, D.Wesołowski pisze:
W dniu 2016-01-05 o 17:28, Tomasz Chiliński pisze:
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L92
jest:
else
winno być:
if ($type == MSG_MAIL)
ponieważ:
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L145 dla wszystkich innych wiadomości innych niż SMS nie stworzy listy odbiorców wiadomości dla np. MSG_WWW, bo no klient nie ma podanego emaila. To samo dla MSG_USERPANEL.
Reasumując, nie podasz w karcie klienta maila to nic do niego nie wyślesz.
aby nie generować wiadomości do klientów nie posiadających komputerów (nam się tacy zdarzają ;) za linią https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L146 dodałbym
. ($type == MSG_WWW ? 'JOIN (SELECT ownerid FROM `nodes` GROUP BY ownerid) cn ON (cn.ownerid = c.id) '
A jeśli chcesz nadać wiadomość www kto nie ma komputera, a np. jakieś urządzenie obsługiwane przez zewnętrzny dodatek? Chyba na razie niech zostanie tak jak jest.
W dniu 2016-01-06 o 14:06, Tomasz Chiliński pisze:
W dniu 06.01.2016 12:55, D.Wesołowski napisał(a):
W dniu 2016-01-06 o 11:31, D.Wesołowski pisze:
W dniu 2016-01-05 o 17:28, Tomasz Chiliński pisze:
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L92
jest:
else
winno być:
if ($type == MSG_MAIL)
ponieważ:
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L145 dla wszystkich innych wiadomości innych niż SMS nie stworzy listy odbiorców wiadomości dla np. MSG_WWW, bo no klient nie ma podanego emaila. To samo dla MSG_USERPANEL.
Reasumując, nie podasz w karcie klienta maila to nic do niego nie wyślesz.
aby nie generować wiadomości do klientów nie posiadających komputerów (nam się tacy zdarzają ;) za linią https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L146 dodałbym
. ($type == MSG_WWW ? 'JOIN (SELECT ownerid FROM `nodes` GROUP BY ownerid) cn ON (cn.ownerid = c.id) '
A jeśli chcesz nadać wiadomość www kto nie ma komputera, a np. jakieś urządzenie obsługiwane przez zewnętrzny dodatek? Chyba na razie niech zostanie tak jak jest.
Tylko tak jak teraz jest, to powoduje np u mnie, że klienci po LTE, czy jakimś innym dostawcy wiszą jako nieodebrane, a po co?
sądzę, że jak ktoś by miał klienta podpiętego po vpn via inny operator to będzie wpisany jako komputer tak czy siak
W dniu 06.01.2016 14:11, D.Wesołowski napisał(a):
W dniu 2016-01-06 o 14:06, Tomasz Chiliński pisze:
W dniu 06.01.2016 12:55, D.Wesołowski napisał(a):
W dniu 2016-01-06 o 11:31, D.Wesołowski pisze:
W dniu 2016-01-05 o 17:28, Tomasz Chiliński pisze:
W dniu 05.01.2016 01:07, D.Wesołowski napisał(a):
Hej,
Cześć,
jaki sens ma tworzenie wiadomości typu www i kierowanie jej tylko do tych klientów, którzy mają wpisanego maila?
Prawdopodobnie jest to pozbawione sensu, ale zanim potwierdzę na 100% to wskaż w pliku z github miejsce, gdzie coś takiego ma miejsce?
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L92
jest:
else
winno być:
if ($type == MSG_MAIL)
ponieważ:
https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L145 dla wszystkich innych wiadomości innych niż SMS nie stworzy listy odbiorców wiadomości dla np. MSG_WWW, bo no klient nie ma podanego emaila. To samo dla MSG_USERPANEL.
Reasumując, nie podasz w karcie klienta maila to nic do niego nie wyślesz.
aby nie generować wiadomości do klientów nie posiadających komputerów (nam się tacy zdarzają ;) za linią https://github.com/lmsgit/lms/blob/master/modules/messageadd.php#L146 dodałbym
. ($type == MSG_WWW ? 'JOIN (SELECT ownerid FROM `nodes` GROUP BY ownerid) cn ON (cn.ownerid = c.id) '
A jeśli chcesz nadać wiadomość www kto nie ma komputera, a np. jakieś urządzenie obsługiwane przez zewnętrzny dodatek? Chyba na razie niech zostanie tak jak jest.
Tylko tak jak teraz jest, to powoduje np u mnie, że klienci po LTE, czy jakimś innym dostawcy wiszą jako nieodebrane, a po co?
sądzę, że jak ktoś by miał klienta podpiętego po vpn via inny operator to będzie wpisany jako komputer tak czy siak
https://github.com/lmsgit/lms/commit/2597360678e101373dc510e3832b7c1c84ec146...
Poszło również cherry-pickiem do stable.
uczestnicy (2)
-
D.Wesołowski -
Tomasz Chiliński