Re: [DISCUSS] Cordova-Android 4.0.0 Release

2015-01-30 Thread Andrew Grieve
Sorry, didn't mean to ignore it. There's no change in that scenario though
(you still need to provide a separate CordovaInterface of you don't extend
CordovaActivity, just as before), except now it's a bit easier to do so,
since you can new leverage the CordovaInterfaceImpl helper class. Does that
make sense?

On Fri, Jan 30, 2015 at 10:45 AM, Joe Bowser bows...@gmail.com wrote:

 You still haven't answered the question about dual inheritance, and how
 people are supposed to use the embedded use case, or you're choosing to
 ignore it.  How are people supposed to inherit from the activity if they
 are already inheriting from another activity, and how are they supposed to
 use a CordovaWebView in that case?


 On Fri Jan 30 2015 at 7:29:39 AM Andrew Grieve agri...@chromium.org
 wrote:

 Just ran them and see no difference in failures with/without change. Did
 need to update two spots which were assuming Activity==Interface, but
 that's kind of the point.

 On Thu, Jan 29, 2015 at 9:39 PM, Joe Bowser bows...@gmail.com wrote:

 And did you run any of the existing tests? This is only the feature that
 caused the JUnit tests to exist in the first place.

 On Thu, Jan 29, 2015, 6:12 PM Andrew Grieve agri...@chromium.org
 wrote:

 On Wed, Jan 28, 2015 at 3:32 PM, Andrew Grieve agri...@chromium.org
 wrote:

 
 
  On Wed, Jan 28, 2015 at 1:44 PM, Joe Bowser bows...@gmail.com
 wrote:
 
  On Wed Jan 28 2015 at 10:38:07 AM Andrew Grieve 
 agri...@chromium.org
  wrote:
 
  
   - Make CordovaActivity not implement CordovaInterface, but instead
  provide
   CordovaInterface via an inner class (to solidify that you can't
 cast the
   activity to CordovaInterface and expect that to work - some used
 to do
  this
   but I think we've cleaned it all up now)
  
   This literally came out of nowhere.  Why are you trying so hard to
  remove
  the embedded view use case? What if someone is implementing an
 activity
  that inherits from another activity like MapActivity?  This API
 change
  came
  without any discussion.
 
  I meant for this to be discussion. Certainly this is non-critical,
 but I
  think it makes the embedded use-case easier not harder. Will do it in
 a PR
  for review.
 

 Here's the issue: https://issues.apache.org/jira/browse/CB-8382
 and PR: https://github.com/apache/cordova-android/pull/152


 
 
 
  All of this can be done in a few days, but I'd also like to see the
 dust
   settle a bit before going forward with 4.0.0 release. E.g. At
 least wait
   until we do a blog post for 3.7.0 (are you doing this?), and have
 done a
   tools release that updates the pinned version to 3.7.0
  
  
  If someone else wants to do the blog post on that, that's fine.  And
 I
  agree that there should be a tools release with 3.7.0 pinned, even
 though
  3.7.0 is really just a technicality so we can get 4.0.0 out IMO.
 
 
  3.7.0 adds Lollipop support. That's pretty big! I won't have time to
 get
  to it this week if there are any other takers?
 
 
 
 
  
   On Wed, Jan 28, 2015 at 12:52 PM, Joe Bowser bows...@gmail.com
 wrote:
  
Reminder: failures with plugins are not blockers.  I've run into
 that
contact issue numerous times when testing with my personal
 device.  I
recommend making sure that your contacts are completely clean so
 that
  you
don't get these weird results.
   
The file failures have been happening for quite a while, and
 those are
   not
blockers for the platform release either.  Do these failures
 happen
  on a
platform other than ICS?
   
On Wed, Jan 28, 2015, 9:06 AM Murat Sutunc 
 mura...@microsoft.com
   wrote:
   
 I’ve ran the mobile-spec tests on android 4.0.3 with 4.0.x and
 there
   are
 some failures. I’ve searched the jira for issues but wasn’t
 able to
   find
 any. Has anyone else ran into these issues before?

 org.apache.cordova.contacts.tests.tests  Contacts
   (navigator.contacts)
 Round trip Contact tests (creating + save + delete + find).
 Contacts.spec.24 Creating, saving, finding a contact should
 work,
  after
 which we should not be able to find it, and we should not be
 able to
delete
 it again.
 •   Expected 2 to be 1
 •   Expected 1 to be 0
  it(contacts.spec.24 Creating, saving, finding a
 contact
   should
 work, removing it should work, after which we should not be
 able to
   find
 it, and we should not be able to delete it again., function
 (done)
  {
   // Save method is not supported on Windows
 platform
   if (isWindows) {
   pending();
   return;
   }
   if (isWindowsPhone8) {
   done();
   return;
   }
   gContactObj = new Contact();
   gContactObj.name = new ContactName();
   gContactObj.name.familyName = DeleteMe;
   

[GitHub] cordova-plugin-test-framework pull request: CB-8388 Prevent Window...

2015-01-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-test-framework/pull/9


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-network-information pull request: CB-8384 Add Windo...

2015-01-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-network-information/pull/24


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-browser pull request: CB-8224: Add support to launch a spe...

2015-01-30 Thread nikhilkh
Github user nikhilkh commented on the pull request:

https://github.com/apache/cordova-browser/pull/8#issuecomment-72233705
  
@stevengill  I noticed you have made some changes in this area. Wondering 
if you can help review this?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Re: Plugin management ?

2015-01-30 Thread John M. Wargo

I published a node module a while back that does it. I originally published it 
as cordova-create, but changed it to cdva-create so I wouldn't disturb anything 
the Cordova team did in the future.

https://www.npmjs.com/package/cdva-create


On 1/22/2015 5:36 AM, Stéphane Wirtel wrote:

Hi all,

In my project, I use a lot of plugins.

So, is there a small tool or a config file where I can specify my dependencies 
(plugins) and just with a command line, install all my plugins ?

I think to grunt (Gruntfile.js), bower (bower.json) and npm (package.json)

Thank you,

Stephane
--
Stéphane Wirtel - http://wirtel.be - @matrixise

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

.





Cordova android native test

2015-01-30 Thread Martin Gonzalez
Hi all,

Recently tested cordova-android master branch and found out a commit that
it's triggering an issue that it got solved a while ago indirectly.
Info:
CB-8317
ccceaeaca
Link:
https://git-wip-us.apache.org/repos/asf?p=cordova-android.git;a=commit;h=ccceaeaca2d72fb9708100ab2f70adb903a27d66
breaks: CB-7371

To avoid future regressions, Joe suggest to create a native test that would
recreate the conditions to verify that the issue is not present.

Here's the deal, the current Cordova Native test suite, with the app
itself, I can't recreate it, MainTestActivity class doesn't get affect at
all, although with a cordova project that uses a single activity it can be
recreate easily.
If I go to any of the other activities and recreate it, that activity gets
destroyed and gets back to the first one(MainTestActivity).
The only way to recreate the problem using that test suite is by using
Error URL (errorurl class) test, which triggers a destroy of the previous
activity caused by not locate the html resource in the path provided.
In that specific test, in that activity and after the previous activity is
closed, in there I can perform the manual test and recreate the problem.

I've added the code to change the configuration and perform a recreation in
that specific test, but that would mean, that it can't go back to the
previous one where all other test live.
Another would be call the activity to perform config changes, pause the
main, and get back to it, apply the configuration in that one and then
trigger the recreation onResume, by listening the configuration changes, or
adding a broadcastreceiver.

Test sample project with a single activity:
--Change the config
--Triger recreation.
Expected: it should recreate the activity
Result (current master): it crash the app.

steps to recreate manually:
1.- Create a project
2.- platform add cordova-android (master)
3.- build  deploy
4.- once launched, minimize it  go to settings
5.- Select another language, and then resume the app, by using the home
button.
At that point crashes

Does anybody have any idea, to perform such as test without having to kill
the main activity?
any other ideas about how to recreate the scenario?

Thanks devs.

-- 
Regards,
Martin Gonzalez


Re: Cordova plugin namespace

2015-01-30 Thread Wargo, John
Before the end of the year, that url referenced a page.  It was a live link, 
not a reference.  

John M. Wargo

 On Jan 28, 2015, at 9:47 AM, Andrew Grieve agri...@chromium.org wrote:
 
 I don't think the URL ever existed. XML namespace URLs don't need to
 actually exist IIRC. They are just identifiers.
 
 On Tue, Jan 27, 2015 at 6:07 PM, John M. Wargo j...@johnwargo.com wrote:
 
 Looking at the plugin guide at http://cordova.apache.org/
 docs/en/4.0.0/plugin_ref_spec.md.html the first sentence on the page
 refers to a Cordova Plugin Namespace page that no longer exists:
 http://apache.org/cordova/ns/plugins/1.0.
 
 Anyone know what happened to it?
 --
 
 *John M. Wargo*
 www.johnwargo.com
 
 

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Re: [DISCUSS] Cordova-Android 4.0.0 Release

2015-01-30 Thread Joe Bowser
You still haven't answered the question about dual inheritance, and how
people are supposed to use the embedded use case, or you're choosing to
ignore it.  How are people supposed to inherit from the activity if they
are already inheriting from another activity, and how are they supposed to
use a CordovaWebView in that case?

On Fri Jan 30 2015 at 7:29:39 AM Andrew Grieve agri...@chromium.org wrote:

 Just ran them and see no difference in failures with/without change. Did
 need to update two spots which were assuming Activity==Interface, but
 that's kind of the point.

 On Thu, Jan 29, 2015 at 9:39 PM, Joe Bowser bows...@gmail.com wrote:

 And did you run any of the existing tests? This is only the feature that
 caused the JUnit tests to exist in the first place.

 On Thu, Jan 29, 2015, 6:12 PM Andrew Grieve agri...@chromium.org wrote:

 On Wed, Jan 28, 2015 at 3:32 PM, Andrew Grieve agri...@chromium.org
 wrote:

 
 
  On Wed, Jan 28, 2015 at 1:44 PM, Joe Bowser bows...@gmail.com wrote:
 
  On Wed Jan 28 2015 at 10:38:07 AM Andrew Grieve agri...@chromium.org
 
  wrote:
 
  
   - Make CordovaActivity not implement CordovaInterface, but instead
  provide
   CordovaInterface via an inner class (to solidify that you can't
 cast the
   activity to CordovaInterface and expect that to work - some used to
 do
  this
   but I think we've cleaned it all up now)
  
   This literally came out of nowhere.  Why are you trying so hard to
  remove
  the embedded view use case? What if someone is implementing an
 activity
  that inherits from another activity like MapActivity?  This API change
  came
  without any discussion.
 
  I meant for this to be discussion. Certainly this is non-critical, but
 I
  think it makes the embedded use-case easier not harder. Will do it in
 a PR
  for review.
 

 Here's the issue: https://issues.apache.org/jira/browse/CB-8382
 and PR: https://github.com/apache/cordova-android/pull/152


 
 
 
  All of this can be done in a few days, but I'd also like to see the
 dust
   settle a bit before going forward with 4.0.0 release. E.g. At least
 wait
   until we do a blog post for 3.7.0 (are you doing this?), and have
 done a
   tools release that updates the pinned version to 3.7.0
  
  
  If someone else wants to do the blog post on that, that's fine.  And I
  agree that there should be a tools release with 3.7.0 pinned, even
 though
  3.7.0 is really just a technicality so we can get 4.0.0 out IMO.
 
 
  3.7.0 adds Lollipop support. That's pretty big! I won't have time to
 get
  to it this week if there are any other takers?
 
 
 
 
  
   On Wed, Jan 28, 2015 at 12:52 PM, Joe Bowser bows...@gmail.com
 wrote:
  
Reminder: failures with plugins are not blockers.  I've run into
 that
contact issue numerous times when testing with my personal
 device.  I
recommend making sure that your contacts are completely clean so
 that
  you
don't get these weird results.
   
The file failures have been happening for quite a while, and
 those are
   not
blockers for the platform release either.  Do these failures
 happen
  on a
platform other than ICS?
   
On Wed, Jan 28, 2015, 9:06 AM Murat Sutunc mura...@microsoft.com
 
   wrote:
   
 I’ve ran the mobile-spec tests on android 4.0.3 with 4.0.x and
 there
   are
 some failures. I’ve searched the jira for issues but wasn’t
 able to
   find
 any. Has anyone else ran into these issues before?

 org.apache.cordova.contacts.tests.tests  Contacts
   (navigator.contacts)
 Round trip Contact tests (creating + save + delete + find).
 Contacts.spec.24 Creating, saving, finding a contact should
 work,
  after
 which we should not be able to find it, and we should not be
 able to
delete
 it again.
 •   Expected 2 to be 1
 •   Expected 1 to be 0
  it(contacts.spec.24 Creating, saving, finding a
 contact
   should
 work, removing it should work, after which we should not be
 able to
   find
 it, and we should not be able to delete it again., function
 (done)
  {
   // Save method is not supported on Windows
 platform
   if (isWindows) {
   pending();
   return;
   }
   if (isWindowsPhone8) {
   done();
   return;
   }
   gContactObj = new Contact();
   gContactObj.name = new ContactName();
   gContactObj.name.familyName = DeleteMe;
   gContactObj.save(function(c_obj) {
   var findWin = function(cs) {
   expect(cs.length).toBe(1);
   // update to have proper saved id
   gContactObj = cs[0];
   gContactObj.remove(function() {
   var findWinAgain = function(seas) {
   

Re: [DISCUSS] Cordova-Android 4.0.0 Release

2015-01-30 Thread Andrew Grieve
Just ran them and see no difference in failures with/without change. Did
need to update two spots which were assuming Activity==Interface, but
that's kind of the point.

On Thu, Jan 29, 2015 at 9:39 PM, Joe Bowser bows...@gmail.com wrote:

 And did you run any of the existing tests? This is only the feature that
 caused the JUnit tests to exist in the first place.

 On Thu, Jan 29, 2015, 6:12 PM Andrew Grieve agri...@chromium.org wrote:

 On Wed, Jan 28, 2015 at 3:32 PM, Andrew Grieve agri...@chromium.org
 wrote:

 
 
  On Wed, Jan 28, 2015 at 1:44 PM, Joe Bowser bows...@gmail.com wrote:
 
  On Wed Jan 28 2015 at 10:38:07 AM Andrew Grieve agri...@chromium.org
  wrote:
 
  
   - Make CordovaActivity not implement CordovaInterface, but instead
  provide
   CordovaInterface via an inner class (to solidify that you can't cast
 the
   activity to CordovaInterface and expect that to work - some used to
 do
  this
   but I think we've cleaned it all up now)
  
   This literally came out of nowhere.  Why are you trying so hard to
  remove
  the embedded view use case? What if someone is implementing an activity
  that inherits from another activity like MapActivity?  This API change
  came
  without any discussion.
 
  I meant for this to be discussion. Certainly this is non-critical, but I
  think it makes the embedded use-case easier not harder. Will do it in a
 PR
  for review.
 

 Here's the issue: https://issues.apache.org/jira/browse/CB-8382
 and PR: https://github.com/apache/cordova-android/pull/152


 
 
 
  All of this can be done in a few days, but I'd also like to see the
 dust
   settle a bit before going forward with 4.0.0 release. E.g. At least
 wait
   until we do a blog post for 3.7.0 (are you doing this?), and have
 done a
   tools release that updates the pinned version to 3.7.0
  
  
  If someone else wants to do the blog post on that, that's fine.  And I
  agree that there should be a tools release with 3.7.0 pinned, even
 though
  3.7.0 is really just a technicality so we can get 4.0.0 out IMO.
 
 
  3.7.0 adds Lollipop support. That's pretty big! I won't have time to get
  to it this week if there are any other takers?
 
 
 
 
  
   On Wed, Jan 28, 2015 at 12:52 PM, Joe Bowser bows...@gmail.com
 wrote:
  
Reminder: failures with plugins are not blockers.  I've run into
 that
contact issue numerous times when testing with my personal
 device.  I
recommend making sure that your contacts are completely clean so
 that
  you
don't get these weird results.
   
The file failures have been happening for quite a while, and those
 are
   not
blockers for the platform release either.  Do these failures happen
  on a
platform other than ICS?
   
On Wed, Jan 28, 2015, 9:06 AM Murat Sutunc mura...@microsoft.com
   wrote:
   
 I’ve ran the mobile-spec tests on android 4.0.3 with 4.0.x and
 there
   are
 some failures. I’ve searched the jira for issues but wasn’t able
 to
   find
 any. Has anyone else ran into these issues before?

 org.apache.cordova.contacts.tests.tests  Contacts
   (navigator.contacts)
 Round trip Contact tests (creating + save + delete + find).
 Contacts.spec.24 Creating, saving, finding a contact should work,
  after
 which we should not be able to find it, and we should not be
 able to
delete
 it again.
 •   Expected 2 to be 1
 •   Expected 1 to be 0
  it(contacts.spec.24 Creating, saving, finding a contact
   should
 work, removing it should work, after which we should not be able
 to
   find
 it, and we should not be able to delete it again., function
 (done)
  {
   // Save method is not supported on Windows platform
   if (isWindows) {
   pending();
   return;
   }
   if (isWindowsPhone8) {
   done();
   return;
   }
   gContactObj = new Contact();
   gContactObj.name = new ContactName();
   gContactObj.name.familyName = DeleteMe;
   gContactObj.save(function(c_obj) {
   var findWin = function(cs) {
   expect(cs.length).toBe(1);
   // update to have proper saved id
   gContactObj = cs[0];
   gContactObj.remove(function() {
   var findWinAgain = function(seas) {
   expect(seas.length).toBe(0);
   gContactObj.remove(function() {
   throw(success callback called
  after
 non-existent Contact object called remove(). Test failed.);
   }, function(e) {
   expect(e.code).toBe(ContactErr
 or.UNKNOWN_ERROR);
   

[GitHub] cordova-plugin-test-framework pull request: CB-8388 Prevent Window...

2015-01-30 Thread nikhilkh
GitHub user nikhilkh opened a pull request:

https://github.com/apache/cordova-plugin-test-framework/pull/9

CB-8388 Prevent Windows app from crashing on test failure



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/MSOpenTech/cordova-plugin-test-framework 
CB-8388

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-test-framework/pull/9.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #9


commit 395d3458a45b162a5aa0278d7fd4d2b322a1ad51
Author: Nikhil Khandelwal nikhi...@microsoft.com
Date:   2015-01-30T16:42:32Z

CB-8388 Prevent Windows app from crashing on test failure




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-file-transfer pull request: CB-5059 Add a CookieMan...

2015-01-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-file-transfer/pull/60


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-android pull request: Make JUnit tests to test pluggable w...

2015-01-30 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-android/pull/140


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-wp8 pull request: Improved console logging

2015-01-30 Thread oliversalzburg
Github user oliversalzburg commented on the pull request:

https://github.com/apache/cordova-wp8/pull/66#issuecomment-72178283
  
@purplecabbage I'll take care of it right away. Thanks for the heads-up.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-network-information pull request: CB-8384 Add Windo...

2015-01-30 Thread sgrebnov
GitHub user sgrebnov opened a pull request:

https://github.com/apache/cordova-plugin-network-information/pull/24

CB-8384 Add Windows platform support to Network Information plugin

https://issues.apache.org/jira/browse/CB-8384

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/sgrebnov/cordova-plugin-network-information 
CB-8384

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-network-information/pull/24.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #24


commit fb608698e5c97e18da8bb1d6f73621bb3676ebfe
Author: sgrebnov v-seg...@microsoft.com
Date:   2015-01-30T11:57:10Z

CB-8384 Add Windows section to Network Information plugin

commit c8281535160a547a367b7c820360e711c59d4cc9
Author: sgrebnov v-seg...@microsoft.com
Date:   2015-01-30T12:33:36Z

CB-8384 Add Windows platform quirks

commit d8a714757fae0ac615224a7a0daabaa88c8e2581
Author: sgrebnov v-seg...@microsoft.com
Date:   2015-01-30T12:37:16Z

CB-8384 Fixes the way we detect online status on Windows




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org