String org.ksoap2.serialization.SoapObject.getPropertySafelyAsString(String name) throws null Exception

manish-kukreja / ksoap2-android

Automatically exported from code.google.com/p/ksoap2-android

0 stars 0 forks source link

String org.ksoap2.serialization.SoapObject.getPropertySafelyAsString(String name) throws null Exception #94

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

What steps will reproduce the problem?

SoapObject p;
...
p.getPropertySafelyAsString("issueSecurityScheme");

And the issueSecurityScheme in envelope body look like this:
 issueSecurityScheme=null; 

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

I should get a string "null" but i get null exception

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

ksoap2-android-assembly-2.6.0-jar-with-dependencies.jar

Please provide any additional information below.

I will check this later and meybe fix this.

In Attach file i give stacktrace

Original issue reported on code.google.com by drozddaw...@gmail.com on 4 Dec 2011 at 5:03

Attachments:

Screenshot-2.png

GoogleCodeExporter commented 9 years ago

Could you provide a fix as a pull request or patch?

Original comment by mosa...@gmail.com on 5 Dec 2011 at 5:32

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

Sure i will try.

Original comment by drozddaw...@gmail.com on 7 Dec 2011 at 12:43

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

[deleted comment]

GoogleCodeExporter commented 9 years ago

The test causes this error. Today i will try fix.(Sorry for comments...)

Original comment by drozddaw...@gmail.com on 8 Dec 2011 at 1:08

Attachments:

SoapObjectTest.java

GoogleCodeExporter commented 9 years ago

Ok i give pull request with fix and add patch here

Original comment by drozddaw...@gmail.com on 8 Dec 2011 at 1:44

Attachments:

0001-Adding-test-to-SoapObject-and-Fix-for-null-pointers.patch

GoogleCodeExporter commented 9 years ago

Patch applied, will go out with 2.6.1 release, let me know if you need it soon

Original comment by mosa...@gmail.com on 8 Dec 2011 at 5:17

GoogleCodeExporter commented 9 years ago

Original comment by mosa...@gmail.com on 8 Dec 2011 at 5:26

Changed state: Fixed

GoogleCodeExporter commented 9 years ago

No problem i make a new jar for my use ;). Sorry for code formatting i forgott 
about it. Next time will be better.

Original comment by drozddaw...@gmail.com on 8 Dec 2011 at 9:47

GoogleCodeExporter commented 9 years ago

Great ;-) I will cut a release when some other fixes are in if somebody 
provides something..

Original comment by mosa...@gmail.com on 8 Dec 2011 at 9:50

GoogleCodeExporter commented 9 years ago

this issue was only fixed in the single arg getPropertySafelyAsString, but it 
still exists in the getPropertySafelyAsString(name, defaultThing) method which 
still tries to call toString without checking for a null return from getProperty

Original comment by jroma...@gmail.com on 17 Feb 2012 at 1:15

GoogleCodeExporter commented 9 years ago

I am opening this again so I dont loose track of the last comment and the 
related problem.

Original comment by mosa...@gmail.com on 30 Apr 2012 at 6:59

Changed state: Accepted

GoogleCodeExporter commented 9 years ago

This second related problem is now fixed in master and should be going out with 
the 2.6.4 release.

Original comment by mosa...@gmail.com on 30 Apr 2012 at 7:23

Changed state: Fixed