[Commotion-dev] Route indicator in links view not updating

[Josh King]

Hi Jason,

Thanks so much for the bug report! Will or Hans, do you have any
comments on whether this is the appropriate behavior? Jason, currently
the best way to contribute is by submitting a patch to the list, or by
filing a bug report at https://code.commotionwireless.net and attaching
a patch file (you can also email bugs at commotionwireless.net to start a
ticket). We are also looking at linking up with Github, so that
contributors can more easily issue pull requests.


Jason Prenger <jason.prenger at datasoft.com> wrote:

>Hey Guys,
>
>Been working with the Android app and got to say this stuff is awesome,
>works great.  I did notice that my HNA indicator wasn't showing up
>properly though in the links list.  I have a small network where every
>node is broadcasting a HNA but only a few would indicators would show
>in
>the links list, and only some of the time.  I could still ping and send
>data to those nodes so I know the routes were being setup properly,
>just
>the indicator wasn't showing correctly some times.
>
>Delved into it a little bit and noticed that in LinksActivity lines 203
>and 204, the new client data's hasRouteToOther and hasDefaultRoute are
>being set to the current (or old) data.  I don't know if that was done
>on purpose or what but it doesn't seem like that should be the case.
>
>I haven't done much work with GIT in the past so I'm not sure the
>procedure for commits or pull-requests but if you want me to go that
>approach let me know.  And if this was on purpose then please disregard
>this
>
>Jason Prenger
>_______________________________________________
>Commotion-dev mailing list
>Commotion-dev at lists.chambana.net
>http://lists.chambana.net/mailman/listinfo/commotion-dev

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 897 bytes
Desc: OpenPGP digital signature
URL: <http://lists.chambana.net/pipermail/commotion-dev/attachments/20120828/c8ef8f7a/attachment.sig>