8000 [DEVX-5006] Add input parameter to 'gather' and coerce language format. · twilio/twilio-python@f754ab4 · GitHub
[go: up one dir, main page]
Skip to content

Commit f754ab4

Browse files
jmctwiliojmctwilio
committed
[DEVX-5006] Add input parameter to 'gather' and coerce language format.
1 parent 21fc89a commit f754ab4

File tree

3 files changed

+43
-2
lines changed

3 files changed

+43
-2
lines changed

‎tests/unit/twiml/__init__.py

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
from nose.tools import raises
44
from six import text_type
55

6-
from twilio.twiml import TwiMLException, TwiML
6+
from twilio.twiml import format_language, TwiMLException, TwiML
77

88

99
class TwilioTest(unittest.TestCase):
@@ -14,3 +14,19 @@ def strip(self, xml):
1414
def test_append_fail(self):
1515
t = TwiML()
1616
t.append('foobar')
17+
18+
def test_format_language_none(self):
19+
language = None
20+
self.assertEqual(language, format_language(language))
21+
22+
def test_format_language_valid(self):
23+
language = 'en-US'
24+
self.assertEqual(language, format_language(language))
25+
26+
def test_format_language_coerced(self):
27+
language = 'EN_us'
28+
self.assertEqual('en-US', format_language(language))
29+
30+
@raises(TwiMLException)
31+
def test_format_language_fail(self):
32+
format_language('this is invalid')

‎twilio/twiml/__init__.py

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import re
12
import xml.etree.ElementTree as ET
23

34

@@ -9,6 +10,21 @@ def lower_camel(string):
910
return result[0].lower() + result[1:]
1011

1112

13+
def format_language(language):
14+
"""
15+
Attempt to format language parameter as 'ww-WW'.
16+
17+
:param string language: language parameter
18+
"""
19+
if not language:
20+
return language
21+
22+
if not re.match('^\w\w[_-]\w\w$', language):
23+
raise TwiMLException('Invalid value for language parameter.')
24+
25+
return language[0:2].lower() + '-' + language[3:5].upper()
26+
27+
1228
class TwiMLException(Exception):
1329
pass
1430

‎twilio/twiml/voice_response.py

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import json
22
from six import string_types
33

4-
from twilio.twiml import TwiML
4+
from twilio.twiml import TwiML, format_language
55

66

77
class VoiceResponse(TwiML):
@@ -111,6 +111,7 @@ def gather(self,
111111
hints=None,
112112
barge_in=None,
113113
acknowledge_sound_url=None,
114+
input=None,
114115
**kwargs):
115116
"""
116117
Add a new <Gather> element
@@ -126,6 +127,7 @@ def gather(self,
126127
:param hints: speech recognition hints
127128
:param barge_in: stop playing media upon speech
128129
:param acknowledge_sound_url: url to hit when sound starts
130+
:param input: type Twilio should accept "dtfm", "speech", "dtfm speech"
129131
:param kwargs: additional attributes
130132
:return: <Gather> element
131133
"""
@@ -135,6 +137,13 @@ def gather(self,
135137
timeout=timeout,
136138
finish_on_key=finish_on_key,
137139
num_digits=num_digits,
140+
partial_result_callback=partial_result_callback,
141+
partial_result_callback_method=partial_result_callback_method,
142+
language=format_language(language),
143+
hints=hints,
144+
barge_in=barge_in,
145+
acknowledge_sound_url=acknowledge_sound_url,
146+
input=input,
138147
**kwargs
139148
))
140149

0 commit comments

Comments
 (0)
0