Przepraszam, ale jakoś byłem przekonany że Cezary się nie myli

Nie da się ukryć, że bez Cezarego i Zuzi to tej apki by nie było.

Ja bym to bardziej nazwał małą niedogodnością, ale oki będę pamiętał o tym przy poprawkach.

Jeszcze Zuzi nie opisałem tego, ale jak tylko będę miał czas aby powalczyć z tematem udostępnię, bez obaw.

Opublikowałem wersję 1.5.

- Dodałem funkcję powiadomienia diodą o nowej wiadomości.
- Poprawiłem zgłoszoną niedogodność dot. wyboru kodu USSD użytkownika.

P.S.
Nie testowałem tej wersji za długo więc gwarancji nie daję, że u wszystkich użytkowników będzie super git.

Informacyjnie przy powiadomieniu o nowej wiadomości:
1a. Najpierw ustawiamy sobie czas sprawdzania (1-9 minut, a i nie ma sensu ustawiać 1 minuty. najlepiej z przedziału 5-9).
1b. Ustawiamy diodę, zapisujemy te ustawienia.
2. Odznaczamy powiadomienie i znów bierzemy zastosuj.

Pokrętnie trochę wiem, ale tak musi być.

Ciesze się Zuziu, że dziala.

Ja juz nie wiem jak mam wrzucac te pliki na githuba, wrzucam na linuksie i tak są bledy w uprawnieniach. Sprawdziłem dla pewności i obydwa te skrypty na dysku mam wykonywalne. Zgłupiałem.

Moim zdaniem warto dodać sprawdzenie, czy coś zgłasza się pod ttyXXX, bo gdy z jakiś powodów modem nie jest dostępny wyskakuje brzydki błąd lua.

Jak przyjdzie SMS wtedy świeci ciągle pulsująco, czyli prawidłowo. Ustawiam sprawdzanie na 5 min. i dioda co 5 min. aktywuje się na 1 sek. i gaśnie, a nie powinna.

@IceG, dobrze idzie, dodatek spełnia założenia. @Cezary pomaga, a ja chętnie przetestuję, bo i tak używam bramki GSM tyle że z Asteriskiem i co ciekawe nie kłóci się z apką.
Co do tego błędu. Najprościej - wyjmij na chwilę modem i otwórz http://192.168.1.1/cgi-bin/luci/admin/modem/sms pojawi się coś takiego:

/usr/lib/lua/luci/template.lua:55: Failed to execute template 'modem/readsms'.
A runtime error occurred: [string "/usr/lib/lua/luci/view/modem/readsms.htm"]:27: attempt to concatenate local 'smscount' (a nil value)
stack traceback:
    [string "/usr/lib/lua/luci/view/modem/readsms.htm"]:27: in main chunk
stack traceback:
    [C]: in function 'n'
    /usr/lib/lua/luci/template.lua:55: in function </usr/lib/lua/luci/template.lua:47>
    (tail call): ?
    /usr/lib/lua/luci/dispatcher.lua:849: in function 'dispatch'
    /usr/lib/lua/luci/dispatcher.lua:833: in function 'dispatch'
    /usr/lib/lua/luci/dispatcher.lua:412: in function </usr/lib/lua/luci/dispatcher.lua:411>

W logu:

Fri Aug 14 12:20:30 2020 daemon.err uhttpd[10670]: open(/dev/ttyUSB0)
Fri Aug 14 12:20:30 2020 daemon.err uhttpd[10670]: tcgetattr(/dev/ttyUSB0)
Fri Aug 14 12:20:30 2020 daemon.err uhttpd[10670]: tcsetattr(/dev/ttyUSB0)
Fri Aug 14 12:20:30 2020 daemon.err uhttpd[10670]: reopen(/dev/ttyUSB0)
Fri Aug 14 12:20:30 2020 daemon.err uhttpd[10670]: failed tcsetattr(/dev/ttyUSB0): Bad file descriptor

Przydałoby się na początku sprawdzanie, czy pod pod ttyXXX coś się zgłasza i komunikat, że nie ma modemu albo coś w tym guście.

@IceG, ale może jakoś prymitywnie:

ls dev/ttyUSB*

jeśli nie ma żadnego urządzenia dodać komunikat, że brak modemu, bo błąd lua nikomu nic nie mówi. Przy wyjętym modemie lub braku sterowników niestety straszy. Tak, wiem, że to nie takie proste

1. Sprawdzić czy /dev/ttyUSB* istnieje
2. Błąd w luci wynikał ze sklejenia pustej wartości z resztą, więc najpierw nalezy sprawdzić czy polecenie/wynik działania zwraca coś sensownego co można pokazać na ekranie.

Jako przykład masz to co wyżej - jeżeli urządzenia nie ma to wynik sprawdzenia ilości sms nic nie zwróci, więc przed wyświetleniem tego na ślepo musisz sprawdzić czy coś jest i dopiero zareagować.

Nie jestem zadowolony bo szczerze mówiąc myślałem, że to już koniec poprawek, grzebania w kodzie i że wszystko działa jak działać powinno, ale nie jest tak pięknie.

Pomału w miarę posiadanego wolnego czasu będę przepisywał kod na wersję 1.6.
1. Do przepisania jest sposób odczytu wiadomości, który teraz jest szybki, ale problematyczny i nie uwzględnia sytuacji z anomaliami modemu. Kolejna miła i czasochłonna zagwostka.

2. Przy powiadomieniu diodą aplikacja na ten moment nie wie, że użytkownik np. mógł usunąć wiadomość, więc to też powinno być dopisane aby skrypty sprawdzały tylko prawidłowy/aktualny stan wiadomości.

Reszta chyba działa Ok.
Aplikacja miała być prosta i miła dla oka (mam nadzieję że taka jest), ale dawno już wyszła poza ramy tego co gdzieś planowałem. A tu ciągle się rozrasta i co gorsza nie widać końca. Ale już mam nadzieję że niedaleko do mety.

Masz Zuziu aktualny skrypt od powiadomień?, u mnie teraz nic nie załącza diody. Testowałem, nawet dziś. Już nie mówię nawet ile $ na smsy wydałem przy tych "zabawach". I nie ustawiaj Zuziu sprawdzania na minutę bo zawsze będziesz zawieszać port.. to nie jest aplikacja wielowątkowa, specjalnie dałem możliwość użytkownikowi dokładnej edycji czasu, aby tak dostosować czas, tak ustawić aby możliwie nie gryzło się z inna odpytującą aplikacją.

Świata nie zbawię, nierealne jest aby wszystko ze wszystkim działało. Ja nie mam Zuziu takiej wiedzy. U mnie np. strona wczytuję się wolniej, od razu widzę że modem jest zajęty, ale Lua nie częstuje błędami.

Rozumiem Zuziu, ale i tak będzie wersja 1.6 bo brakuje teraz tak jak pisałem wyżej, a bez przepisana tego inaczej nie zrobię. Skoro działa już to postaram się dużej rewolucji nie robić, tylko dodam to co brakuje bez odpytywania o dostępność portów/portu.