Poprawki do bramki Orange

Marek Madej mmadej w gmail.com
Śro, 9 Wrz 2009, 11:38:59 CEST


Witam,

To mój pierwszy list na liście, więc chcę się ładnie przywitać :-)

Użytkuję program sms od dobrych kilku lat, ale dopiero niedawno odkryłem
istnienie tej listy (zatrzymałem się na jakiejś starej wersji,
utrzymywanej jeszcze przez inicjatora programu, samodzielnie
wprowadzając poprawki do interesującego mnie kodu, czyli bramki Orange).
Swego czasu na bazie tego kodu stworzyłem moduł SMS orangemultibox do
kadu, jednak brak czasu nie pozwolił mi na przystosowanie go do nowego
API kadu i projekt modułu niestety umarł...

Jest kilka rzeczy, które można zdecydowanie poprawić. Po pierwsze,
bramka Orange pozwala na wysłanie SMSa do DOWOLNEJ sieci - nie tylko
polskiej, więc nie ma potrzeby "obcinać" numeru telefonu z trzech
pierwszych znaków (czyli +48), bo tak naprawdę może tam być na przykład
+44 albo +49. Stąd mój pierwszy, trywialny patch sms-2.2.5-allnumbers.patch

Sprawa kolejna: w Multiboksie istnieje opcja dołączenia we własnym
profilu numeru stacjonarnego - wówczas bramka pozwala na wysłanie 10
SMSów na numery stacjonarne. Jednak bieżąca wersja programu nie
obsługuje niestety takiej bramki (niepoprawnie parsuje informacje o
ilości bezpłatnych i płatnych SMSów). Dodałem więc obsługę ilości SMSów
stacjonarnych - program zachowuje się poprawnie zarówno dla "normalnych"
profili Multibox, jak i dla tych z numerem stacjonarnym. To drugi patch:
sms-2.2.5-fixedlinembox.patch

Sam używam programu z kodowaniem utf8 a nie iso, stąd trzeci patch:
sms-2.2.5-utf8.patch
oraz wersja utf8 patcha drugiego.

Wszystkie patche są tutaj:
http://najnowsze.info/repo/sms/
bonus - prosty PKGBUILD dla Archa

Jeżeli tylko czas mi pozwoli, chciałbym w przyszłości zaproponować kilka
usprawnień (poprzez przygotowanie odpowiednich patchy):
- obsługa gettext (czyli: uniezależnienie się od kodowania znaków oraz
możliwość tłumaczenia na obce języki)
- optymalizacja kodu (najpierw dla bramki Orange: sporo kodu jest na
zasadzie copy&paste),
- optymalizacja użycia bramki Orange: można zmniejszyć ilość żądań HTTP
o jednego "geta" poprzez modyfikację wartości parametru
loginSuccessfulURL: po zalogowaniu od razu dostaniemy stronę bramki, z
której można przeczytać ilość SMSów. To nieco "śliski" temat, bo admini
mogą zauważyć w logach niestandardowe logowania i ukrócić taki proceder,
ale z drugiej strony zdecydowanie skraca się czas wysyłania SMSa.

Szczerze powiedziawszy nie wiem, kto zajmuje się "wydawaniem" i
numerowaniem kolejnych wersji. Myślę, że te dwa moje usprawnienia
zasługują na 2.2.6 i mam nadzieję, że przejdą pomyślną weryfikację.

Pozdrawiam,
Marek





Więcej informacji o liście Sms-users