poprawki do modulu voip
Ta poprawka dodaje do modulu voip 2 dodatkowe pola. Pole komentarza - ktorego przydatnosci z pewnoscia nie trzeba tlumaczyc oraz dodatkowe pole na haslo do tkzw portalu. W moim przypadku portalem jest panel abonencji na centrali gdzie klienci moga sobie sprawdzac swoje polaczenia. Przypuszczam ze taki lub inny portal kazdy ma wiec i takie pole bedzie przydatne.
Pierwszy raz robie patcha wiec moze byc cos zrobione zle albo bez sensu wiec prosze o sugestie jak to ew. poprawic.
Jaroslaw Czarniak wrote:
Ta poprawka dodaje do modulu voip 2 dodatkowe pola. Pole komentarza - ktorego przydatnosci z pewnoscia nie trzeba tlumaczyc oraz dodatkowe pole na haslo do tkzw portalu. W moim przypadku portalem jest panel abonencji na centrali gdzie klienci moga sobie sprawdzac swoje polaczenia. Przypuszczam ze taki lub inny portal kazdy ma wiec i takie pole bedzie przydatne.
Mam dwie uwagi. Ja bym kolumny nazwał uipasswd (panel) i description (comments). Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
Pierwszy raz robie patcha wiec moze byc cos zrobione zle albo bez sensu wiec prosze o sugestie jak to ew. poprawic.
Prosiłbym patcha w jednym pliku (diff -r). Szybciej się przegląda i aplikuje.
Monday 16 February 2009 15:04:17 A.L.E.C napisał(a):
Mam dwie uwagi. Ja bym kolumny nazwał uipasswd (panel) i description (comments).
To zaden problem. Poprawie.
Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
To dostosowalem tylko do tego co juz bylo. Czyli hasla sie pokazywaly juz wczesniej ja jedynie dodalem takie samo rozwiazanie. do pozostalych pol.
Prosiłbym patcha w jednym pliku (diff -r). Szybciej się przegląda i aplikuje.
Sprobuje.
Jaroslaw Czarniak wrote:
Monday 16 February 2009 15:04:17 A.L.E.C napisał(a):
Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
To dostosowalem tylko do tego co juz bylo. Czyli hasla sie pokazywaly juz wczesniej ja jedynie dodalem takie samo rozwiazanie. do pozostalych pol.
Jakoś mi to umknęło wcześniej, ale podtrzymuję swoje zdanie, jakbyś mógł to zrób z tym porządek.
On Mon, 16 Feb 2009 18:28:08 +0100, A.L.E.C wrote
Jaroslaw Czarniak wrote:
Monday 16 February 2009 15:04:17 A.L.E.C napisał(a):
Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
To dostosowalem tylko do tego co juz bylo. Czyli hasla sie pokazywaly juz wczesniej ja jedynie dodalem takie samo rozwiazanie. do pozostalych pol.
Jakoś mi to umknęło wcześniej, ale podtrzymuję swoje zdanie, jakbyś mógł to zrób z tym porządek.
To nie jest moim zdaniem dobry pomysł. Zapominasz hasło abonenta i co? Kupa?
Pozdrawiam, Tomek.
On Mon, 16 Feb 2009 22:19:50 +0100, Tomasz Chiliński wrote
On Mon, 16 Feb 2009 18:28:08 +0100, A.L.E.C wrote
Jaroslaw Czarniak wrote:
Monday 16 February 2009 15:04:17 A.L.E.C napisał(a):
Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
To dostosowalem tylko do tego co juz bylo. Czyli hasla sie pokazywaly juz wczesniej ja jedynie dodalem takie samo rozwiazanie. do pozostalych pol.
Jakoś mi to umknęło wcześniej, ale podtrzymuję swoje zdanie, jakbyś mógł to zrób z tym porządek.
To nie jest moim zdaniem dobry pomysł. Zapominasz hasło abonenta i co? Kupa?
Oczywiście miałem na myśli wariant w którym abonent zapomina swoje hasło. Odczytywanie tego w SQŁ-u to lipa.
Pozdrawiam.
On Mon, 16 Feb 2009 22:19:50 +0100, Tomasz Chiliński wrote
On Mon, 16 Feb 2009 18:28:08 +0100, A.L.E.C wrote
Jaroslaw Czarniak wrote:
Monday 16 February 2009 15:04:17 A.L.E.C napisał(a):
Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
To dostosowalem tylko do tego co juz bylo. Czyli hasla sie pokazywaly juz wczesniej ja jedynie dodalem takie samo rozwiazanie. do pozostalych pol.
Jakoś mi to umknęło wcześniej, ale podtrzymuję swoje zdanie, jakbyś mógł to zrób z tym porządek.
To nie jest moim zdaniem dobry pomysł. Zapominasz hasło abonenta i co? Kupa?
Pozdrawiam, Tomek.
lms mailing list lms@lists.lms.org.pl http://lists.lms.org.pl/mailman/listinfo/lms
I bardzo dobrze że zapominasz hasła. Jedyna okazja żeby ustawić klientowi nowe. Popieram pomysł aby hasła dla VOIP były zakodowane. Administrator nie musi haseł klientów znać ani pamiętać w żadnym razie.
pozdrawiam Dariusz Kowalczyk
!DSPAM:4999f9a7157075315134984!
Dariusz Kowalczyk - Webvisor wrote:
I bardzo dobrze że zapominasz hasła. Jedyna okazja żeby ustawić klientowi nowe. Popieram pomysł aby hasła dla VOIP były zakodowane. Administrator nie musi haseł klientów znać ani pamiętać w żadnym razie.
Nie chodziło mi o samo kodowanie haseł ale o pokazywanie ich na liście. Natomiast jeśli o samo kodowanie chodzi to w sumie myślę, że więcej plusów ma rozwiązanie z zakodowanymi hasłami (choć nie zawsze można takie hasła stosować, zależy od tego do czego je potrzebujemy). Dlatego w hostingu mamy kodowane.
Monday 16 February 2009 15:04:17 A.L.E.C napisał(a):
Mam dwie uwagi. Ja bym kolumny nazwał uipasswd (panel) i description (comments). Nie podoba mi się też pomysł pokazywania haseł na liście kont, zostawiłbym na niej tylko login, numer i opis.
Poprawione
Prosiłbym patcha w jednym pliku (diff -r). Szybciej się przegląda i aplikuje.
Nie wiem jak zrobic pacza aby dzialal z glownego katalogu wiec polaczenie ich jest bez sensu...
Jaroslaw Czarniak wrote:
Nie wiem jak zrobic pacza aby dzialal z glownego katalogu wiec polaczenie ich jest bez sensu...
najprościej jest przed wprowadzaniem zmian skopiować cały katalog LMSa i teraz masz jeden katalog "źródłowy" i jeden z poprawkami, polecenie robiące diffa zbiorczego wygląda następująco:
diff -ruNa stary-katalog nowy-katalog > patch.file
to samo można zrobić z jednym katalogiem przy pomocy "cvs diff".
Patcha należy też przejrzeć i usunąć z niego niepotrzebne rzeczy takie jak np. zawartość templates_c.
Jeszcze mam do tych zmian jedną uwagę o której przypomniał lexx. Jeżeli zmiany zawierają nowe kolumny, tabele itp. w strukturze bazy danych, dobrze byłoby gdybyś dodał odpowiednie zmiany w plikach DDL (doc/lms.*) oraz w procedurze auto-upgrade (lib/upgradedb/*).
Tuesday 17 February 2009 10:20:21 A.L.E.C napisał(a):
Jeszcze mam do tych zmian jedną uwagę o której przypomniał lexx. Jeżeli zmiany zawierają nowe kolumny, tabele itp. w strukturze bazy danych, dobrze byłoby gdybyś dodał odpowiednie zmiany w plikach DDL (doc/lms.*) oraz w procedurze auto-upgrade (lib/upgradedb/*).
Jeszcze jedna mala poprawka - dodalem generowanie hasel.
uczestnicy (4)
-
A.L.E.C -
Dariusz Kowalczyk - Webvisor -
Jaroslaw Czarniak -
Tomasz Chiliński