Here's the log: https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=log;h=97ad4d84ce333f05d6240b75f65da8536bcc3eb1
On Wed, Dec 11, 2013 at 1:22 PM, Axel Nennker <ignisvul...@gmail.com> wrote: > 3.3 will be released tomorrow, right? > > -Axel > > (Are there more patches put in after rc1?) > > > 2013/12/11 Joe Bowser <bows...@gmail.com> >> >> It appears that your issue has been fixed by the changes made in >> CB-5481. This patch was put in AFTER rc1 was released. >> https://issues.apache.org/jira/browse/CB-5481 >> >> On Wed, Dec 11, 2013 at 12:44 PM, Axel Nennker <ignisvul...@gmail.com> >> wrote: >> > Here is the issue on JIRA: >> > https://issues.apache.org/jira/browse/CB-5633 >> > >> > This code works with 3.3-rc1: >> > >> > public void loadPlugins() { >> > Activity activity = this.ctx.getActivity(); >> > Log.d(this.getClass().getName(), >> > >> > "activity.getClass().getPackage().getName()="+activity.getClass().getPackage().getName()); >> > Intent intent = >> > >> > activity.getPackageManager().getLaunchIntentForPackage(activity.getPackageName()); >> > Log.d(this.getClass().getName(), >> > "intent.getPackage()="+intent.getPackage()); >> > int id = activity.getResources().getIdentifier("config", "xml", >> > intent.getPackage()); >> > if (id == 0) { >> > >> > -Axel >> > >> > >> > >> > 2013/12/11 Joe Bowser <bows...@gmail.com> >> >> >> >> Your bug doesn't exist unless it has a JIRA issue. >> >> >> >> On Wed, Dec 11, 2013 at 12:25 PM, Axel Nennker <ignisvul...@gmail.com> >> >> wrote: >> >> > Jesse, >> >> > I did not create an JIRA issue yet because I do not have a small >> >> > sample >> >> > project to reproduce this. >> >> > I am using the command line to create this cordova app. >> >> > The project has a config.xml and it starts correctly (which is prove >> >> > that it >> >> > has the config.xml. otherwise I would see the spinner eternally). >> >> > The version is 3.3-rc1 (Installed 30 minutes ago). >> >> > >> >> > I have an ant script that does the work. I would not 100%-ly rule it >> >> > out >> >> > that this build.xml is missing some steps. It creates the projects >> >> > folders >> >> > (www, merges, etc) and then calls e.g. cordova platform add android >> >> > and >> >> > cordova build. "cordova create hello com.example.hello HelloWorld" is >> >> > NOT >> >> > called. >> >> > I attached my build.xml and the AndroidManifest.xml >> >> > >> >> > -Axel >> >> > >> >> > config.xml is there where it belongs. >> >> > >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ ls -l >> >> > platforms/android/res/xml/ >> >> > insgesamt 4 >> >> > -rwxrwxrwx 1 root root 2528 Dez 11 20:22 config.xml >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ >> >> > >> >> > >> >> > >> >> > >> >> > >> >> > 2013/12/11 Jesse <purplecabb...@gmail.com> >> >> >> >> >> >> Is this a JIRA issue and reproducable steps yet? >> >> >> Does the project that is failing actually have a config.xml file? >> >> >> How are you creating the project? Platform script, cordova-cli? What >> >> >> version of the cli? ... >> >> >> >> >> >> 3.3.0 is already tagged and ready to go, so this needs reproducible >> >> >> steps >> >> >> to get attention. >> >> >> >> >> >> >> >> >> @purplecabbage >> >> >> risingj.com >> >> >> >> >> >> >> >> >> On Wed, Dec 11, 2013 at 11:28 AM, Axel Nennker >> >> >> <ignisvul...@gmail.com>wrote: >> >> >> >> >> >> > Sorry no. It does not work on 3.3-rc1 >> >> >> > Axel >> >> >> > >> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ cordova -v >> >> >> > 3.3.0-rc.1 >> >> >> > ignisvulpis@ubuntu:/host/20131210/phonegap$ >> >> >> > >> >> >> > same error: >> >> >> > D/CordovaActivity( 4871): CordovaActivity.init() >> >> >> > D/CordovaWebView( 4871): >>> >> >> >> > >> >> >> > >> >> >> > >> >> >> > >> >> >> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json) >> >> >> > D/PluginManager( 4871): init() >> >> >> > E/PluginManager( 4871): >> >> >> > >> >> >> > >> >> >> > >> >> >> > >> >> >> > ===================================================================================== >> >> >> > E/PluginManager( 4871): ERROR: config.xml is missing. Add >> >> >> > res/xml/config.xml to your project. >> >> >> > E/PluginManager( 4871): >> >> >> > >> >> >> > >> >> >> > >> >> >> > >> >> >> > https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=blob;f=framework/res/xml/config.xml >> >> >> > E/PluginManager( 4871): >> >> >> > >> >> >> > >> >> >> > >> >> >> > >> >> >> > ===================================================================================== >> >> >> > D/CordovaWebView( 4871): >>> loadUrlNow() >> >> >> > D/ItemImport( 4871): onResume() loadURL started: >> >> >> > D/CordovaActivity( 4871): >> >> >> > >> >> >> > >> >> >> > >> >> >> > >> >> >> > onMessage(onPageStarted,file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-MyCard2-1.json) >> >> >> > >> >> >> > >> >> >> > >> >> >> > >> >> >> > 2013/12/11 Andrew Grieve <agri...@chromium.org> >> >> >> > >> >> >> > > If it worked in 3.1 and broke in 3.2, then it's very likely >> >> >> > > CB-5481<https://issues.apache.org/jira/browse/CB-5481> and >> >> >> > > is fixed in 3.3 >> >> >> > > >> >> >> > > >> >> >> > > On Wed, Dec 11, 2013 at 11:29 AM, Joe Bowser <bows...@gmail.com> >> >> >> > > wrote: >> >> >> > > >> >> >> > > > No. As far as I'm concerned this problem doesn't exist until >> >> >> > > > it >> >> >> > > > is a >> >> >> > JIRA >> >> >> > > > issue with some sample code that I can use to reproduce the >> >> >> > > > error. I >> >> >> > > doubt >> >> >> > > > that this was a regression since we never explicitly supported >> >> >> > > > your >> >> >> > > > use >> >> >> > > > case to begin with. >> >> >> > > > >> >> >> > > > Of course, patches are always welcome. >> >> >> > > > On 11 Dec 2013 07:58, "Axel Nennker" <ignisvul...@gmail.com> >> >> >> > > > wrote: >> >> >> > > > >> >> >> > > > > No. I intented this to be on the list. Gmail UI irks. >> >> >> > > > > Could this be fixed in 3.3? >> >> >> > > > > I know it is late but... >> >> >> > > > > >> >> >> > > > > -Axel >> >> >> > > > > >> >> >> > > > > >> >> >> > > > > 2013/12/11 Joe Bowser <bows...@gmail.com> >> >> >> > > > > >> >> >> > > > > > Did you mean to to send this to me personally? >> >> >> > > > > > >> >> >> > > > > > On Wed, Dec 11, 2013 at 7:24 AM, Axel Nennker < >> >> >> > ignisvul...@gmail.com >> >> >> > > > >> >> >> > > > > > wrote: >> >> >> > > > > > > Well, this repeatable. Use 3.2 fail, go back to 3.1 win. >> >> >> > > > > > > The config.xml is where it belongs but PluginManager >> >> >> > > > > > > does >> >> >> > > > > > > not >> >> >> > find >> >> >> > > it >> >> >> > > > > in >> >> >> > > > > > a >> >> >> > > > > > > certain (legal) setting. >> >> >> > > > > > > When the app is started "normally" PluginManager finds >> >> >> > > > > > > config.xml >> >> >> > > > > (which >> >> >> > > > > > > proves that is at the correct location) >> >> >> > > > > > > When the app is started because an IntentFilter triggers >> >> >> > > > > > > then >> >> >> > > > > > > PluginInManager does not find it. >> >> >> > > > > > > >> >> >> > > > > > > The "main" activity is in one class like a.b.c.Main >> >> >> > > > > > > while >> >> >> > > > > > > the >> >> >> > > > > > IntentFilter >> >> >> > > > > > > activity is in another e.g. u.v.w.IF >> >> >> > > > > > > IF starts Main successfully but - my guess is - that >> >> >> > PluginManager >> >> >> > > > > tries >> >> >> > > > > > to >> >> >> > > > > > > find config.xml in package u.v.w >> >> >> > > > > > > >> >> >> > > > > > > The JUNIT/spec test should be updated to catch this >> >> >> > > > > > > situation >> >> >> > > > > > > if >> >> >> > > > > > possible. >> >> >> > > > > > > I guess I could change the package names so that IF is >> >> >> > > > > > > in >> >> >> > > > > > > a.b.c >> >> >> > too >> >> >> > > > but >> >> >> > > > > > it >> >> >> > > > > > > worked in 3.1 and I see no reason why this is "complex" >> >> >> > > > > > > The u.v.w.IF features are implemented in a plugin that >> >> >> > > > > > > registers >> >> >> > > the >> >> >> > > > > > intent >> >> >> > > > > > > filter in the app's AndroidManifest. It is developed by >> >> >> > > > > > > my >> >> >> > > colleagues >> >> >> > > > > who >> >> >> > > > > > > chose the u.v.w package name. I am using this plugin in >> >> >> > > > > > > my >> >> >> > > > > > > app >> >> >> > that >> >> >> > > > has >> >> >> > > > > > the >> >> >> > > > > > > a.b.c package namespace. >> >> >> > > > > > > I do not agree that this a complex or irregular use of >> >> >> > > > > > > cordova >> >> >> > > > > > > or >> >> >> > > > > > Intents. >> >> >> > > > > > > >> >> >> > > > > > > Please revert the code back to the 3.1 behaviour. >> >> >> > > > > > > >> >> >> > > > > > > Axel >> >> >> > > > > > > >> >> >> > > > > > > >> >> >> > > > > > > 2013/12/11 Joe Bowser <bows...@gmail.com> >> >> >> > > > > > >> >> >> >> > > > > > >> I haven't noticed this during any of our testing, nor >> >> >> > > > > > >> have >> >> >> > > > > > >> I >> >> >> > > noticed >> >> >> > > > > > >> this when hacking with Google Glass, which uses the >> >> >> > > > > > >> Google >> >> >> > > > > > >> Glass >> >> >> > > > voice >> >> >> > > > > > >> trigger intent to launch Cordova, not the LAUNCHER >> >> >> > > > > > >> intent >> >> >> > > > > > >> like >> >> >> > the >> >> >> > > > > > >> launch menu. Furthermore, our JUnit tests haven't >> >> >> > > > > > >> caught >> >> >> > > > > > >> the >> >> >> > > tests >> >> >> > > > > > >> either, and they depend on at least three plugins to be >> >> >> > > > > > >> loaded >> >> >> > by >> >> >> > > > > > >> PluginManager from config.xml. Honestly, this only >> >> >> > > > > > >> breaks >> >> >> > > > > > >> if >> >> >> > you >> >> >> > > > > > >> don't know what you're doing with Intents. >> >> >> > > > > > >> >> >> >> > > > > > >> Honestly, It sounds to me like your config.xml isn't in >> >> >> > > > > > >> the >> >> >> > > correct >> >> >> > > > > > >> place in your project, especially if you're trying to >> >> >> > > > > > >> do >> >> >> > something >> >> >> > > > > > >> complex with Android and intents. I would check that >> >> >> > > > > > >> making >> >> >> > these >> >> >> > > > > > >> sorts of statements. If config.xml wasn't being read, >> >> >> > > > > > >> it >> >> >> > > > > > >> would >> >> >> > > have >> >> >> > > > > > >> been caught right away, since everything is a plugin >> >> >> > > > > > >> now. >> >> >> > > > > > >> >> >> >> > > > > > >> >> >> >> > > > > > >> >> >> >> > > > > > >> On Wed, Dec 11, 2013 at 4:33 AM, >> >> >> > > > > > >> <axel.nenn...@telekom.de> >> >> >> > > wrote: >> >> >> > > > > > >> > Hi, >> >> >> > > > > > >> > >> >> >> > > > > > >> > We are seeing some strange effect on Android when we >> >> >> > > > > > >> > move >> >> >> > > > > > >> > from >> >> >> > > > > > >> > 3.1.0-0.2.0 to a newer version. >> >> >> > > > > > >> > We tried with 3.2.0-0.4.0 and 3.3-rc1 >> >> >> > > > > > >> > >> >> >> > > > > > >> > I think that PluginManager changed and now fails to >> >> >> > > > > > >> > load >> >> >> > > > config.xml >> >> >> > > > > > >> > >> >> >> > > > > > >> > This happens in my project when the app is opened by >> >> >> > > > > > >> > the >> >> >> > > operating >> >> >> > > > > > >> > system because it is configured so through an intent >> >> >> > > > > > >> > filter. >> >> >> > > > > > >> > The original activity is not a child of DroidGap but >> >> >> > > > > > >> > starts >> >> >> > the >> >> >> > > > > "main" >> >> >> > > > > > >> > activity using startActivity. >> >> >> > > > > > >> > >> >> >> > > > > > >> > The code to get the id of the config.xml file is not >> >> >> > > > > > >> > working >> >> >> > in >> >> >> > > > this >> >> >> > > > > > >> > case. >> >> >> > > > > > >> > >> >> >> > > > > > >> > Why was the 3.1.0 code changed? >> >> >> > > > > > >> > >> >> >> > > > > > >> > Cheers >> >> >> > > > > > >> > Axel >> >> >> > > > > > >> > >> >> >> > > > > > >> > D/CordovaActivity(12320): CordovaActivity.init() >> >> >> > > > > > >> > D/CordovaWebView(12320): >>> >> >> >> > > > > > >> > >> >> >> > > > > > >> >> >> > > > > >> >> >> > > > >> >> >> > > >> >> >> > >> >> >> > >> >> >> > >> >> >> > loadUrl(file:///android_asset/www/index.html?file=/storage/sdcard0/Download/WalletImport-InfoCard-PremiumBank2-2.json) >> >> >> > > > > > >> > D/PluginManager(12320): init() >> >> >> > > > > > >> > E/PluginManager(12320): >> >> >> > > > > > >> > >> >> >> > > > > > >> >> >> > > > > >> >> >> > > > >> >> >> > > >> >> >> > >> >> >> > >> >> >> > >> >> >> > ===================================================================================== >> >> >> > > > > > >> > E/PluginManager(12320): ERROR: config.xml is missing. >> >> >> > > > > > >> > Add >> >> >> > > > > > >> > res/xml/config.xml to your project. >> >> >> > > > > > >> > E/PluginManager(12320): >> >> >> > > > > > >> > >> >> >> > > > > > >> >> >> > > > > >> >> >> > > > >> >> >> > > >> >> >> > >> >> >> > >> >> >> > >> >> >> > https://git-wip-us.apache.org/repos/asf?p=incubator-cordova-android.git;a=blob;f=framework/res/xml/plugins.xml >> >> >> > > > > > >> > E/PluginManager(12320): >> >> >> > > > > > >> > >> >> >> > > > > > >> >> >> > > > > >> >> >> > > > >> >> >> > > >> >> >> > >> >> >> > >> >> >> > >> >> >> > ===================================================================================== >> >> >> > > > > > >> > D/CordovaWebView(12320): >>> loadUrlNow() >> >> >> > > > > > >> > D/ItemImport(12320): onResume() loadURL started: >> >> >> > > > > > > >> >> >> > > > > > > >> >> >> > > > > > >> >> >> > > > > >> >> >> > > > >> >> >> > > >> >> >> > >> >> > >> >> > >> > >> > > >
