kingdom578 / maven-android-plugin

Automatically exported from code.google.com/p/maven-android-plugin
0 stars 0 forks source link

Please write code in common sense #426

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
OK, write the steps to reproduce the issue as normal

1. Change the android app's manifest file's root node as "<manifest 
xml:a='http://schemas.android.com/apk/res/android'"
2.Update all of the prefix "android:" to "a:"
3.Setup the app with our lovely maven-android-plugin by goal android:run.
Excepted result: App is set up correctly
Actual result: The activity can not be launched.

We can find the exception log by reset up with -e option in maven .
There's issue in the RunMojo.execute()

We can easily find your source in line 243 of 
com.jayway.maven.plugins.android.standalonemojos.RunMojo.java is:
String activityName = activities.item( 0 ).getAttributes().getNamedItem( 
"android:name" ).getNodeValue();

Come on guys, could you please unmarshall the xml string in common? 

And also there's a problem it show a error as this:
Activity class {xxx.xxx.xxx.xxx/xxx.xxx.xxx.xxx.MainActivity} does not exist. 
It really exist, I don't know why this error there. I think it's the same 
reason.

Original issue reported on code.google.com by palmtal...@gmail.com on 31 Oct 2013 at 5:00

GoogleCodeExporter commented 9 years ago
Feel free to provide a pull request with a fix

Original comment by mosa...@gmail.com on 18 Dec 2013 at 7:10

GoogleCodeExporter commented 9 years ago

Original comment by mosa...@gmail.com on 31 Dec 2013 at 7:01