Skip to content

Commit

Permalink
make access to add_optional_parameter easier
Browse files Browse the repository at this point in the history
  • Loading branch information
smn committed Jan 7, 2014
1 parent ffd2e79 commit ece5bd3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
1 change: 1 addition & 0 deletions .gitignore
Expand Up @@ -8,3 +8,4 @@ coverage-html/
coverage.xml
nosetests.xml
pep8.txt
dist/
11 changes: 5 additions & 6 deletions smpp/pdu_builder.py
Expand Up @@ -15,8 +15,7 @@ def __init__(self,
self.obj['header']['command_status'] = command_status
self.obj['header']['sequence_number'] = sequence_number


def __add_optional_parameter(self, tag, value):
def add_optional_parameter(self, tag, value):
if self.obj.get('body') == None:
self.obj['body'] = {}
if self.obj['body'].get('optional_parameters') == None:
Expand All @@ -28,13 +27,13 @@ def __add_optional_parameter(self, tag, value):
})

def set_sar_msg_ref_num(self, value):
self.__add_optional_parameter('sar_msg_ref_num', value)
self.add_optional_parameter('sar_msg_ref_num', value)

def set_sar_segment_seqnum(self, value):
self.__add_optional_parameter('sar_segment_seqnum', value)
self.add_optional_parameter('sar_segment_seqnum', value)

def set_sar_total_segments(self, value):
self.__add_optional_parameter('sar_total_segments', value)
self.add_optional_parameter('sar_total_segments', value)


def get_obj(self):
Expand Down Expand Up @@ -200,7 +199,7 @@ def __init__(self,
def add_message_payload(self, value):
self.obj['body']['mandatory_parameters']['sm_length'] = 0
self.obj['body']['mandatory_parameters']['short_message'] = None
self._PDU__add_optional_parameter('message_payload', value)
self.add_optional_parameter('message_payload', value)


class SubmitMulti(SM1):
Expand Down

0 comments on commit ece5bd3

Please sign in to comment.