search

DRB-IT / blacktiger

Blacktiger server code written in Java.
2 stars 1 forks source link

Updater står som UNKNOWN på telefonopkald #77

Closed paf61 closed 10 years ago

paf61 commented 10 years ago

Når en lytter ringer ind fra telefonnettet, skrives der ”UNKNOWN” i feltet updater i databasen. Det var tanken at der skal stå ID’et på den mødesal der blev ringet til, fx ”H45-9000-1”. Det er SQL procedure get_call_info der skal have oplysningen som den anden parameter.

paf61 commented 10 years ago

Måske er mødesal ID ikke tilgængelig her, hvis det ikke er webfrontenden der kalder get_call_info? I så fald indsæt det kaldte nummer i stedet for "< UNKNOWN >".

michaelkrog commented 10 years ago

@paf61 Er det her stadig et problem i 2.0.4(der pt. er på dev1.telesal.org)?

paf61 commented 10 years ago

Der står ikke længere UNKNOWN, men der er opstået et andet problem: Efter jeg har lavet et opkald fra en telefon der ikke før har været ringet fra, ses nummeret slet ikke på listen DEBUG_show_all_callers. Først når jeg skiver et navn ud for nummeret, gemmes det i db.

Det var hensigten at GET_CALL_INFO (som også skriver i db) skal kaldes straks et opkald kommer ind. Outputtet fra GET_CALL_INFO skal så bruges til at afgøre om opkaldet kom fra en sal, en telefon eller en sip-lytter, så det kan vises korrekt (bla. ikon) på websiden. Det er ikke tanken at blacktiger selv skal tjekke numrene for at finde ud af hvilken type opkald det er, blacktiger skal bruge parameteren H, P eller C fra GET_HALL_INFO så vi har en ren snitflade her.

Jeg tror oprindeligt det var vores tanke at GET_CALL_INFO skulle kaldes fra web (altså kun når salen er logget på), med det vil jo måske være mere rigtigt at den kaldes af backenden (blactiger, ikke -web) ligesom WRITE_CALL_LOG? Men så kan vi ikke udfylde feltet nr. 2, "indlogget sal", med sip-id, men det kunne så i stedet være salens telefonnummer, den oplysning er alligevel kun til debug.

michaelkrog commented 10 years ago

Hmm. Mystisk..

Jeg kalder GET_CALL_INFO så snart der kommer et nyt event om at en ny deltager er ankommet i et møderum(ParticipantJoinEvent). De info der kommer fra asterisk dekoreres med info der kan hentes fra GET_CALL_INFO.

Jeg prøvede lige lokalt og sletter jeg info om nummer jeg ringer ind fra og derefter ringer op, så dukker der automatisk en ny række op med det nummer.

Jeg bruger dog ganske vist ikke typen fra GET_HALL_INFO - den finder jeg pt. selv frem. Grunden til at jeg har gjort det er at det kan være lidt svært at få de stored procedures til passe ind i en objekt orienteret struktur, hvilket betyder at jeg nogle steder skal "kortslutte" strukturen for at få data ind de rigtige steder.

Men jeg laver det om nu, så typen altid er fra de stored procedures.