<div dir="ltr"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small;color:rgb(0,0,153)"><span style="font-family:arial;color:rgb(34,34,34)">On Mon, Jan 20, 2014 at 4:17 PM, Frederick Grose </span><span dir="ltr" style="font-family:arial;color:rgb(34,34,34)"><<a href="mailto:fgrose@gmail.com" target="_blank">fgrose@gmail.com</a>></span><span style="font-family:arial;color:rgb(34,34,34)"> wrote:</span><br>

</div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Fixes #4410  <a href="https://bugs.sugarlabs.org/ticket/4410" target="_blank">https://bugs.sugarlabs.org/ticket/4410</a><br>


<br>
The 'Discard network history' button is peculiar to Sugar, and<br>
is destructive to all non-wireless connections, despite being in<br>
the 'Wireless' section of the control panel.<br>
<br>
In the proposed patch[0], the code variables and functions are<br>
renamed and coded to be specific to wireless connections.<br>
<br>
The 'Radio' checkbox instruction and 'Discard ...' button are<br>
reworded to avoid confusion over their state and actions.[1]<br>
<br>
If wireless hardware is not present on the system, the 'Wireless'<br>
section of the control panel is excluded.[2]<br>
<br>
[0,1,2] - <a href="https://github.com/sugarlabs/sugar/pull/228" target="_blank">https://github.com/sugarlabs/sugar/pull/228</a><br>
</blockquote></div><br></div><div class="gmail_extra"><font color="#000000"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small;color:rgb(0,0,153);display:inline"></div></font><span style="color:rgb(0,0,0);font-family:'trebuchet ms',sans-serif">Daniel Narvaez</span><div class="gmail_default" style="color:rgb(0,0,0);font-family:'trebuchet ms',sans-serif;display:inline">

 wrote via</div><span style="color:rgb(0,0,0);font-family:'trebuchet ms',sans-serif"> </span><span style="color:rgb(0,0,0);font-family:'trebuchet ms',sans-serif"><a href="mailto:notifications@github.com">notifications@github.com</a></span></div>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="color:rgb(0,0,0);font-family:arial,sans-serif;font-size:13px">It seems like the code is handling adhoc networks specia</span><span style="font-size:13px;font-family:arial,sans-serif">lly, but I didn't find an explanation of that change. Can you please elaborate on it?</span><span style="color:rgb(0,0,153);font-family:'trebuchet ms',sans-serif"></span></blockquote>

<div><br></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small"><font color="#000000">The current code at,</font></div><div class="gmail_extra"><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small">

<font color="#000000"><a href="https://github.com/sugarlabs/sugar/blob/master/src/jarabe/model/network.py#L835">https://github.com/sugarlabs/sugar/blob/master/src/jarabe/model/network.py#L835</a></font></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small">

<font color="#000000"><br></font></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small"><font color="#000000">tests for sugar-internal connections by reading the connection_id string.</font></div>

<div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small"><font color="#000000"><br></font></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small">

<font color="#000000">The new code </font><span style="color:rgb(0,0,0)">is specific to wireless connections.</span></div><div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small"><span style="color:rgb(0,0,0)">The first condition, the presence of a wifi_settings list, avoids 7 of the 7 sugar-internal</span></div>

<div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small"><span style="color:rgb(0,0,0)">connections on XO-1.  </span><span style="color:rgb(0,0,0)">The sugar ad hoc connections are avoided by a second condition,</span></div>

<div class="gmail_default" style="font-family:'trebuchet ms',sans-serif;font-size:small"><span style="color:rgb(0,0,0)">the same test as in the currrent code, </span><span style="color:rgb(0,0,0)">connection_id.startswith(ADHOC_CONNECTION_ID_PREFIX).</span></div>

<br></div></div>