* [PATCH 2/6] stk-api.txt: Describe new d-bus APIs related to BIP commands. @ 2011-03-22 12:51 Philippe Nunes 2011-03-24 1:26 ` Denis Kenzior 0 siblings, 1 reply; 4+ messages in thread From: Philippe Nunes @ 2011-03-22 12:51 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 3641 bytes --] --- doc/stk-api.txt | 27 ++++++++++++++++++++++++++- 1 files changed, 26 insertions(+), 1 deletions(-) diff --git a/doc/stk-api.txt b/doc/stk-api.txt index 529c734..772e4ce 100644 --- a/doc/stk-api.txt +++ b/doc/stk-api.txt @@ -96,6 +96,7 @@ Methods byte RequestSelection(string title, byte icon_id, an error given below. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession Implementation notes: @@ -111,6 +112,7 @@ Methods byte RequestSelection(string title, byte icon_id, cleared prior to the display of this text. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession [service].Error.SimToolkit.Busy Implementation notes: @@ -154,6 +156,7 @@ Methods byte RequestSelection(string title, byte icon_id, should be opaque. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession Implementation notes: @@ -171,6 +174,7 @@ Methods byte RequestSelection(string title, byte icon_id, are expected. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession string RequestKey(string alpha, byte icon_id) @@ -179,6 +183,7 @@ Methods byte RequestSelection(string title, byte icon_id, for the request. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession string RequestDigit(string alpha, byte icon_id) @@ -186,12 +191,14 @@ Methods byte RequestSelection(string title, byte icon_id, expected. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession boolean RequestConfirmation(string alpha, byte icon_id) Asks the agent to get confirmation from the user. Possible Errors: [service].Error.SimToolkit.GoBack + [service].Error.SimToolkit.EndSession boolean ConfirmCallSetup(string information, byte icon_id) @@ -245,6 +252,7 @@ Methods byte RequestSelection(string title, byte icon_id, Possible Errors: [service].Error.SimToolkit.EndSession void DisplayActionInformation(string text, byte icon_id) + [noreply] Supplies a text string and/or icon concerning the current activity in the terminal and UICC. The @@ -260,6 +268,23 @@ Methods byte RequestSelection(string title, byte icon_id, confirmation message to oFono and then should open the launch browser with the given url. + boolean ConfirmOpenChannel(string information, byte icon_id) + + Asks the agent to request user to confirm the channel + set-up. + + Possible Errors: [service].Error.SimToolkit.EndSession + + void DisplayChannelActivity(string text, byte icon_id) + + Supplies a text string and/or icon concerning the + current activity linked to the channel set-up. The + text should be displayed to the user on screen + until the call is canceled using Cancel() or until the + user decides to end the session. + + Possible Errors: [service].Error.SimToolkit.EndSession + void Cancel() [noreply] Asks the agent to cancel any ongoing operation in @@ -270,6 +295,6 @@ Methods byte RequestSelection(string title, byte icon_id, void Release() [noreply] Agent is being released, possibly because of oFono - terminating, SimToolkit interface torn down or modem + terminating, SimToolkit interface turn down or modem off. If the agent is registered as a global agent, no UnregisterAgent call is expected. -- 1.7.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 2/6] stk-api.txt: Describe new d-bus APIs related to BIP commands. 2011-03-22 12:51 [PATCH 2/6] stk-api.txt: Describe new d-bus APIs related to BIP commands Philippe Nunes @ 2011-03-24 1:26 ` Denis Kenzior 2011-03-24 14:34 ` Philippe Nunes 0 siblings, 1 reply; 4+ messages in thread From: Denis Kenzior @ 2011-03-24 1:26 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 4346 bytes --] Hi Philippe, On 03/22/2011 07:51 AM, Philippe Nunes wrote: > --- > doc/stk-api.txt | 27 ++++++++++++++++++++++++++- > 1 files changed, 26 insertions(+), 1 deletions(-) > > diff --git a/doc/stk-api.txt b/doc/stk-api.txt > index 529c734..772e4ce 100644 > --- a/doc/stk-api.txt > +++ b/doc/stk-api.txt > @@ -96,6 +96,7 @@ Methods byte RequestSelection(string title, byte icon_id, > an error given below. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > > Implementation notes: > > @@ -111,6 +112,7 @@ Methods byte RequestSelection(string title, byte icon_id, > cleared prior to the display of this text. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > [service].Error.SimToolkit.Busy > > Implementation notes: > @@ -154,6 +156,7 @@ Methods byte RequestSelection(string title, byte icon_id, > should be opaque. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > > Implementation notes: > > @@ -171,6 +174,7 @@ Methods byte RequestSelection(string title, byte icon_id, > are expected. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > > string RequestKey(string alpha, byte icon_id) > > @@ -179,6 +183,7 @@ Methods byte RequestSelection(string title, byte icon_id, > for the request. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > > string RequestDigit(string alpha, byte icon_id) > > @@ -186,12 +191,14 @@ Methods byte RequestSelection(string title, byte icon_id, > expected. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > > boolean RequestConfirmation(string alpha, byte icon_id) > > Asks the agent to get confirmation from the user. > > Possible Errors: [service].Error.SimToolkit.GoBack > + [service].Error.SimToolkit.EndSession > That's great that you're fixing up the possible errors in the documentation, but they really need to be in a separate patch. > boolean ConfirmCallSetup(string information, byte icon_id) > > @@ -245,6 +252,7 @@ Methods byte RequestSelection(string title, byte icon_id, > Possible Errors: [service].Error.SimToolkit.EndSession > > void DisplayActionInformation(string text, byte icon_id) > + [noreply] > Why are you marking this noreply when we do actually expect one? > Supplies a text string and/or icon concerning the > current activity in the terminal and UICC. The > @@ -260,6 +268,23 @@ Methods byte RequestSelection(string title, byte icon_id, > confirmation message to oFono and then should open > the launch browser with the given url. > > + boolean ConfirmOpenChannel(string information, byte icon_id) > + > + Asks the agent to request user to confirm the channel > + set-up. > + > + Possible Errors: [service].Error.SimToolkit.EndSession > + > + void DisplayChannelActivity(string text, byte icon_id) > + > + Supplies a text string and/or icon concerning the > + current activity linked to the channel set-up. The > + text should be displayed to the user on screen > + until the call is canceled using Cancel() or until the > + user decides to end the session. > + Why can we not re-use DisplayActionInformation here? > + Possible Errors: [service].Error.SimToolkit.EndSession > + > void Cancel() [noreply] > > Asks the agent to cancel any ongoing operation in > @@ -270,6 +295,6 @@ Methods byte RequestSelection(string title, byte icon_id, > void Release() [noreply] > > Agent is being released, possibly because of oFono > - terminating, SimToolkit interface torn down or modem > + terminating, SimToolkit interface turn down or modem The original is actually correct, but should probably have the word 'being' in front of torn. e.g. SimToolkit interface is being torn down or modem terminating... > off. If the agent is registered as a global agent, > no UnregisterAgent call is expected. Regards, -Denis ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/6] stk-api.txt: Describe new d-bus APIs related to BIP commands. 2011-03-24 1:26 ` Denis Kenzior @ 2011-03-24 14:34 ` Philippe Nunes 2011-03-24 15:26 ` Denis Kenzior 0 siblings, 1 reply; 4+ messages in thread From: Philippe Nunes @ 2011-03-24 14:34 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 5670 bytes --] On 03/24/2011 02:26 AM, Denis Kenzior wrote: > Hi Philippe, > > On 03/22/2011 07:51 AM, Philippe Nunes wrote: >> --- >> doc/stk-api.txt | 27 ++++++++++++++++++++++++++- >> 1 files changed, 26 insertions(+), 1 deletions(-) >> >> diff --git a/doc/stk-api.txt b/doc/stk-api.txt >> index 529c734..772e4ce 100644 >> --- a/doc/stk-api.txt >> +++ b/doc/stk-api.txt >> @@ -96,6 +96,7 @@ Methods byte RequestSelection(string title, byte icon_id, >> an error given below. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> >> Implementation notes: >> >> @@ -111,6 +112,7 @@ Methods byte RequestSelection(string title, byte icon_id, >> cleared prior to the display of this text. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> [service].Error.SimToolkit.Busy >> >> Implementation notes: >> @@ -154,6 +156,7 @@ Methods byte RequestSelection(string title, byte icon_id, >> should be opaque. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> >> Implementation notes: >> >> @@ -171,6 +174,7 @@ Methods byte RequestSelection(string title, byte icon_id, >> are expected. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> >> string RequestKey(string alpha, byte icon_id) >> >> @@ -179,6 +183,7 @@ Methods byte RequestSelection(string title, byte icon_id, >> for the request. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> >> string RequestDigit(string alpha, byte icon_id) >> >> @@ -186,12 +191,14 @@ Methods byte RequestSelection(string title, byte icon_id, >> expected. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> >> boolean RequestConfirmation(string alpha, byte icon_id) >> >> Asks the agent to get confirmation from the user. >> >> Possible Errors: [service].Error.SimToolkit.GoBack >> + [service].Error.SimToolkit.EndSession >> > > That's great that you're fixing up the possible errors in the > documentation, but they really need to be in a separate patch. OK, so be it. I will split this patch > >> boolean ConfirmCallSetup(string information, byte icon_id) >> >> @@ -245,6 +252,7 @@ Methods byte RequestSelection(string title, byte icon_id, >> Possible Errors: [service].Error.SimToolkit.EndSession >> >> void DisplayActionInformation(string text, byte icon_id) >> + [noreply] >> > > Why are you marking this noreply when we do actually expect one? In practice, this API is called once an alpha Id is provided by the proactive commands SEND SS, SEND SMS, SEND USSD. For such proactive commands, no "user" answer is expected. That's why the error code [service].Error.SimToolkit.EndSession is even not allowed. And that's the reason why I added the criteria [No reply] Now, I just realized that I missed the proactive command SEND DTMF for which this API is also relevant and for which the user can indeed request to end the session. So, the criteria [noreply] is indeed not fair. But, the reason for which I introduced the new API DisplayChannelActivity was to distinguish precisely when the STK agent could allow the user to end the session (typically when the button Cancel could be displayed). Now, to handle correctly the case SEND DTMF, I presume that it would be better to refactor this API 'DisplayActionInformation' in order to address all the proactive commands which require simply to display an alpha ID. But this requires I guess to introduce a new argument like a boolean "end_session_allowed". > >> Supplies a text string and/or icon concerning the >> current activity in the terminal and UICC. The >> @@ -260,6 +268,23 @@ Methods byte RequestSelection(string title, byte icon_id, >> confirmation message to oFono and then should open >> the launch browser with the given url. >> >> + boolean ConfirmOpenChannel(string information, byte icon_id) >> + >> + Asks the agent to request user to confirm the channel >> + set-up. >> + >> + Possible Errors: [service].Error.SimToolkit.EndSession >> + >> + void DisplayChannelActivity(string text, byte icon_id) >> + >> + Supplies a text string and/or icon concerning the >> + current activity linked to the channel set-up. The >> + text should be displayed to the user on screen >> + until the call is canceled using Cancel() or until the >> + user decides to end the session. >> + > > Why can we not re-use DisplayActionInformation here? > >> + Possible Errors: [service].Error.SimToolkit.EndSession >> + >> void Cancel() [noreply] >> >> Asks the agent to cancel any ongoing operation in >> @@ -270,6 +295,6 @@ Methods byte RequestSelection(string title, byte icon_id, >> void Release() [noreply] >> >> Agent is being released, possibly because of oFono >> - terminating, SimToolkit interface torn down or modem >> + terminating, SimToolkit interface turn down or modem > > The original is actually correct, but should probably have the word > 'being' in front of torn. e.g. SimToolkit interface is being torn down > or modem terminating... > >> off. If the agent is registered as a global agent, >> no UnregisterAgent call is expected. > > Regards, > -Denis > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 2/6] stk-api.txt: Describe new d-bus APIs related to BIP commands. 2011-03-24 14:34 ` Philippe Nunes @ 2011-03-24 15:26 ` Denis Kenzior 0 siblings, 0 replies; 4+ messages in thread From: Denis Kenzior @ 2011-03-24 15:26 UTC (permalink / raw) To: ofono [-- Attachment #1: Type: text/plain, Size: 2369 bytes --] Hi Philippe, >>> boolean ConfirmCallSetup(string information, byte icon_id) >>> >>> @@ -245,6 +252,7 @@ Methods byte RequestSelection(string >>> title, byte icon_id, >>> Possible Errors: [service].Error.SimToolkit.EndSession >>> >>> void DisplayActionInformation(string text, byte icon_id) >>> + [noreply] >>> >> >> Why are you marking this noreply when we do actually expect one? > > In practice, this API is called once an alpha Id is provided by the > proactive commands SEND SS, SEND SMS, SEND USSD. > > For such proactive commands, no "user" answer is expected. That's why > the error code [service].Error.SimToolkit.EndSession is even not > allowed. And that's the reason why I added the criteria [No reply] Ok fair enough. > > Now, I just realized that I missed the proactive command SEND DTMF for > which this API is also relevant and for which the user can indeed > request to end the session. So, the criteria [noreply] is indeed not fair. > Send DTMF is a little bit more subtle. The problem is that it is usually handled by the modem side and we have no control over the terminal response. So the current Send DTMF implementation will likely work magically everywhere anyway. We probably should split up this implementation into two paths: - If the command is notified via ofono_stk_proactive_command_handled_notify, then do as we do now, e.g. using DisplayActionInformation - If the command is notified via ofono_stk_proactive_command_notify, then we should probably call a different function. > But, the reason for which I introduced the new API > DisplayChannelActivity was to distinguish precisely when the STK agent > could allow the user to end the session (typically when the button > Cancel could be displayed). > And that makes sense to me now > Now, to handle correctly the case SEND DTMF, I presume that it would be > better to refactor this API 'DisplayActionInformation' in order to > address all the proactive commands which require simply to display an > alpha ID. But this requires I guess to introduce a new argument like a > boolean "end_session_allowed". > I think a separate function makes sense. Perhaps DisplayAbortableActionInfo or feel free to come up with a better name. Regards, -Denis ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-03-24 15:26 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2011-03-22 12:51 [PATCH 2/6] stk-api.txt: Describe new d-bus APIs related to BIP commands Philippe Nunes 2011-03-24 1:26 ` Denis Kenzior 2011-03-24 14:34 ` Philippe Nunes 2011-03-24 15:26 ` Denis Kenzior
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.