alexchengalan / csipsimple

Automatically exported from code.google.com/p/csipsimple
0 stars 0 forks source link

Cannot erase Proxy URI field once it has been filled in #79

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Create an account using the Expert Wizard
2. Verify that it connects to the SIP server
3. Re-Edit the account and add a Proxy URI (e.g. same domain as Registration 
URI)
4. Verify that it still connects
5. Re-Edit the account and blank out the Proxy URI field
6. Close the Expert Wizard
7. Re-edit the account and open the Proxy URI item

What is the expected output? What do you see instead?

Expected: blank, no proxy name
Saw: The previous proxy name before it was blanked out

What version of the product are you using? On what operating system?

0.00-12 (pre5) SVN rev 134  Android 2.1 Update 1 HTC Incredible

Please provide any additional information below.

Nothing further. I will look into this one now...

Original issue reported on code.google.com by dc3de...@gmail.com on 9 Jul 2010 at 10:16

GoogleCodeExporter commented 9 years ago
Wasn't looking at the proxy count before saving the proxy string into the 
database. In DBAdapter.java:

// Assume we have an unique proxy
if (account.cfg.getProxy_cnt() > 0)
    args.put(FIELD_PROXY, account.cfg.getProxy().getPtr());
else
    args.put(FIELD_PROXY, (String)null);

Original comment by dc3de...@gmail.com on 9 Jul 2010 at 10:34

GoogleCodeExporter commented 9 years ago
Fixed in 0.00-12.

Original comment by r3gis...@gmail.com on 10 Jul 2010 at 2:36