<p>herlesupreeth has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/pysim/+/16946">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Initial implementation for programming MSISDN<br><br>Change-Id: Ie033a0ffc3697ae562eaa7a241a0f6af6c2b0594<br>---<br>M pySim-prog.py<br>1 file changed, 22 insertions(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/46/16946/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/pySim-prog.py b/pySim-prog.py</span><br><span>index 2638eef..739c7b1 100755</span><br><span>--- a/pySim-prog.py</span><br><span>+++ b/pySim-prog.py</span><br><span>@@ -157,6 +157,10 @@</span><br><span>                      help="Perform a 'dry run', don't actually program the card",</span><br><span>                   default=False, action="store_true")</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+     parser.add_option("--msisdn", dest="msisdn",</span><br><span style="color: hsl(120, 100%, 40%);">+                      help="MSISDN",</span><br><span style="color: hsl(120, 100%, 40%);">+              )</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>  (options, args) = parser.parse_args()</span><br><span> </span><br><span>    if options.type == 'list':</span><br><span>@@ -262,6 +266,22 @@</span><br><span>    # Digitize MCC/MNC (5 or 6 digits)</span><br><span>   plmn_digits = _mcc_mnc_digits(mcc, mnc)</span><br><span> </span><br><span style="color: hsl(120, 100%, 40%);">+   if opts.name is not None and len(opts.name) > 16:</span><br><span style="color: hsl(120, 100%, 40%);">+          raise ValueError('Service Provider Name must max 16 characters!');</span><br><span style="color: hsl(120, 100%, 40%);">+            name = opts.name</span><br><span style="color: hsl(120, 100%, 40%);">+      else:</span><br><span style="color: hsl(120, 100%, 40%);">+         name = None</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+ if opts.msisdn is not None:</span><br><span style="color: hsl(120, 100%, 40%);">+           if not _isnum(opts.msisdn):</span><br><span style="color: hsl(120, 100%, 40%);">+                   raise ValueError('MSISDN must be digits only!');</span><br><span style="color: hsl(120, 100%, 40%);">+              if len(opts.msisdn) > 10*2:</span><br><span style="color: hsl(120, 100%, 40%);">+                        # Need to support MSISDN of length > 20</span><br><span style="color: hsl(120, 100%, 40%);">+                    raise ValueError('For now support for MSISDN of length <= 20 digits!');</span><br><span style="color: hsl(120, 100%, 40%);">+            msisdn = opts.msisdn</span><br><span style="color: hsl(120, 100%, 40%);">+  else:</span><br><span style="color: hsl(120, 100%, 40%);">+         msisdn = None</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span>      # ICCID (19 digits, E.118), though some phase1 vendors use 20 :(</span><br><span>     if opts.iccid is not None:</span><br><span>           iccid = opts.iccid</span><br><span>@@ -390,7 +410,7 @@</span><br><span> </span><br><span>         # Return that</span><br><span>        return {</span><br><span style="color: hsl(0, 100%, 40%);">-                'name'  : opts.name,</span><br><span style="color: hsl(120, 100%, 40%);">+          'name'  : name,</span><br><span>              'iccid' : iccid,</span><br><span>             'mcc'   : mcc,</span><br><span>               'mnc'   : mnc,</span><br><span>@@ -400,6 +420,7 @@</span><br><span>                 'opc'   : opc,</span><br><span>               'acc'   : acc,</span><br><span>               'pin_adm' : pin_adm,</span><br><span style="color: hsl(120, 100%, 40%);">+          'msisdn' : msisdn,</span><br><span>   }</span><br><span> </span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/pysim/+/16946">change 16946</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/pysim/+/16946"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: pysim </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ie033a0ffc3697ae562eaa7a241a0f6af6c2b0594 </div>
<div style="display:none"> Gerrit-Change-Number: 16946 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: herlesupreeth <herlesupreeth@gmail.com> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>