I personally think it's fine as it is. Bada does the same thing. Not all methods need to have the same signature. Only the common ones do. WebOS might support features that are not supported on other platforms and users might still want access to those. commandProxy is just another way of doing what webworks/bada/ripple are doing…it doesn't address this particular issue.
On Wed, Oct 24, 2012 at 6:59 AM, Leutwyler, Markus <[email protected]>wrote: > Could you elaborate a bit? Which functions should be rewritten? Also, > would that have to be done for 2.2.0rc2 already? > > Thanks > > Markus > > -----Original Message----- > From: Jesse MacFadyen [mailto:[email protected]] > Sent: Dienstag, 23. Oktober 2012 14:53 > To: [email protected] > Subject: Re: cordova-js webOS > > Look at windows8 and commandProxy > > Cheers, > Jesse > > Sent from my iPhone6 > > On 2012-10-23, at 4:52 AM, "Leutwyler, Markus" <[email protected]> > wrote: > > > Correct, I'm not using exec marshaling for all the functions. Is there > something that needs to be changed in the code? > > > > Thanks > > > > Markus > > > > -----Original Message----- > > From: Herm Wong [mailto:[email protected]] > > Sent: Montag, 22. Oktober 2012 23:56 > > To: [email protected] > > Subject: RE: cordova-js webOS > > > > Got it figured out. > > The webOS specific methods defined in the plugin/webos directory need to > use the navigator.classname.method to invoke the APIs for the class. > > It looks like overlapping methods from the common/plugin will get > overridden by plugin/webos For example: > > // seems to override common/plugin/notification alert method with > > plugin/webos/Notification alert methodvar args = new Array();args[0] = > > "testing alert";args[1] = null;args[2] = "title";args[3] = > > "blah";Cordova.exec(null, null, "Notification", "alert", args); // > > showBanner doesn't exist in common/plugin notification - so I used the > > cordova-webos/phonegap-webos syntax to call the showBanner > > methodnavigator.notification.showBanner('test message'); > > > >> From: [email protected] > >> To: [email protected] > >> Date: Mon, 22 Oct 2012 14:30:38 -0700 > >> Subject: Re: cordova-js webOS > >> > >> The invocation from exec: > >> > >> https://github.com/apache/incubator-cordova-js/blob/master/lib/webos/ > >> e > >> xec.j > >> s#L50 > >> > >> > >> Which is `method(successCallback, failCallback, args)` > >> > >> Does not match _any_ of the webos notification method function > signatures: > >> > >> https://github.com/apache/incubator-cordova-js/blob/master/lib/webos/ > >> p > >> lugin > >> /webos/notification.js#L48 > >> > >> > >> Ie. showBanner: function(message, response, icon, soundClass, > >> soundFile, > >> soundDurationMs) > >> > >> On 10/22/12 2:26 PM, "Herm Wong" <[email protected]> wrote: > >> > >>> Yeap. Tried that as well. It's looking like the exec call is being > >>> made but the banner isn't being displayed. The palm-log isn't > >>> displaying any errors. > >>> Will continue to debug the issue. > >>> > >>>> From: [email protected] > >>>> To: [email protected] > >>>> Date: Mon, 22 Oct 2012 14:24:02 -0700 > >>>> Subject: Re: cordova-js webOS > >>>> > >>>> > >>>> https://github.com/apache/incubator-cordova-js/blob/master/lib/webo > >>>> s > >>>> /exec > >>>> .j > >>>> s#L43 > >>>> > >>>> > >>>> The notification API is defined in exec with uppercase Notification > >>>> in webOS' case. Try "Notification" as the param into exec, as per > >>>> the common notification plugin: > >>>> > >>>> > >>>> https://github.com/apache/incubator-cordova-js/blob/master/lib/comm > >>>> o > >>>> n/plu > >>>> gi > >>>> n/notification.js#L41 > >>>> > >>>> > >>>> On 10/22/12 1:41 PM, "Herm Wong" <[email protected]> wrote: > >>>> > >>>>> The webOS cordova-js will boot up properly and fire the > >>>>> deviceReady > >>>> event > >>>>> with Gord's & Anis' changes merged. > >>>>> However I'm still having troubles getting the APIs to work. > >>>>> For example: > >>>>> Cordova.exec(null, null, "notification", "showBanner", "hello > >>>>> world"); returns the following error in the palm log: > >>>>> [20121022-11:47:05.902046] > >>>>> error: TypeError: Cannot call method 'showBanner' of undefined, > >>>>> cordova.webos.js:951 > >>>>> It looks like the plugin mapping isn't able to find the > >>>>> notification class since it's undefined. > >>>>> I also tried testing the cordova.webos.js file with the webOS > >>>>> kitchen sink sample app and wasn't able to get the app to > >>>>> work.https://github.com/hermwong/PhoneGap-webOS-kitchen-sink-app > >>>>> We may want to tag cordova-js 2.2.0rc2 and release it without > >>>> cordova-js > >>>>> webOS until we're able to track down & resolve this issue. > >>>>> > >>>>>> From: [email protected] > >>>>>> To: [email protected] > >>>>>> Subject: RE: cordova-js webOS > >>>>>> Date: Sat, 20 Oct 2012 09:07:15 -0700 > >>>>>> > >>>>>> I'll test it Monday when I'm back in SF (where my webOS devices > >>>>>> are located). > >>>>>> > >>>>>>> From: [email protected] > >>>>>>> Date: Fri, 19 Oct 2012 22:17:41 -0400 > >>>>>>> Subject: Re: cordova-js webOS > >>>>>>> To: [email protected] > >>>>>>> > >>>>>>> That wouldn't have caught it, the main problem was code that > >>>>>>> wasn't indented to the correct level, check out the diff to see > >>>>>>> what I > >>>> did. > >>>>>>> > >>>>>>> > >>>>>>> On Fri, Oct 19, 2012 at 8:28 PM, Anis KADRI > >>>>>>> <[email protected]> > >>>>>> wrote: > >>>>>>> > >>>>>>>> I, myself, just called fixwhitespace task. Isn't that supposed > >>>>>>>> to > >>>>>> fix the > >>>>>>>> indentation issues ? Jake stopped barking after I did it, so I > >>>>>> assumed > >>>>>>>> everything was alright. > >>>>>>>> > >>>>>>>> On Fri, Oct 19, 2012 at 4:44 PM, Gord Tanner > >>>> <[email protected]> > >>>>>> wrote: > >>>>>>>> > >>>>>>>>> Hey, > >>>>>>>>> > >>>>>>>>> I took a look at the webos code and it had a number of > >>>>>> indentation issues > >>>>>>>>> and some other papercuts that I cleaned up on a personal > >>>> branch. > >>>> > >>>>>> https://github.com/gtanner/incubator-cordova-js/commit/d123863e08 > >>>>>> 9 > >>>>>> dc5f9 > >>>>>> 98 > >>>>>> d68c2867d2bc17636262ab > >>>>>>>>> > >>>>>>>>> I don't have a webos device to test that everything still > >>>>>>>>> works > >>>>>> but it > >>>>>>>>> should since most of what I fixed was just cleaning up the > >>>>>>>>> code > >>>>>>>> formatting > >>>>>>>>> issues. > >>>>>>>>> > >>>>>>>>> Could someone with a webos device do a quick test of this > >>>> branch > >>>>>> to see > >>>>>>>> if > >>>>>>>>> it still works and we can get this merged in to master. > >>>>>>>>> > >>>>>>>>> Thanks, > >>>>>>>>> > >>>>>>>>> Gord > >>>>>>>>> > >>>>>>>>> On Fri, Oct 12, 2012 at 12:02 PM, Josh Soref <[email protected]> > >>>>>> wrote: > >>>>>>>>> > >>>>>>>>>> Markus wrote: > >>>>>>>>>>> Fixed, every file has now a new line at the end > >>>>>>>>>> > >>>>>>>>>> As a vcs purity note, your fix for this had an unrelated > >>>> change > >>>>>> in the > >>>>>>>>>> commit: > >>>> > >>>>>> https://github.com/ghtomcat/incubator-cordova-js/commit/3acbb8d992 > >>>>>> 07bd9 > >>>>>> b1 > >>>>>> 163b71734f4e999bd4d6512#L1L23 > >>>>>>>>>> > >>>>>>>>>> @@ -20,7 +20,6 @@ var plugins = { > >>>>>>>>>> - /*"File" : require('cordova/plugin/webos/filereader'),*/ > >>>>>>>>>> > >>>>>>>>>> @@ -35,4 +34,4 @@ module.exports = function(success, > >>>>>>>>>> fail, > >>>>>> service, > >>>>>>>>>> action, args) { > >>>>>>>>>> -}; > >>>>>>>>>> \ No newline at end of file > >>>>>>>>>> +}; > >>>>>>>>>> > >>>>>>>>>> Personally, I'd rewrite all the changesets: > >>>>>>>>>> * folding in the newlines into the original commit > >>>>>>>>>> * possibly keeping the indentation consistency thing as > >>>>>>>>>> a > >>>>>> distinct > >>>>>>>>> commit > >>>>>>>>>> * definitely splitting the comment removal into its own > >>>> commit > >>>>>> if it is > >>>>>>>>>> meaningful, or folding it into the original commit otherwise. > >>>>>>>>>> > >>>>>>>>>> -- but this is easy for me to do with mercurial and > >>>>>>>>>> would > >>>>>> probably > >>>>>>>> result > >>>>>>>>>> in me pulling more hairs with git -- and it would give > >>>>>>>>>> me the > >>>>>> pain of > >>>>>>>>>> trying to figure out if I need to close/open new pull > >>>> requests > >>>>>> with > >>>>>>>>> github > >>>>>>>>>> whose ui inevitably trips me up. > >>>>>> ----------------------------------------------------------------- > >>>>>> ---- > >>>>>>>>>> This transmission (including any attachments) may > >>>>>>>>>> contain > >>>>>> confidential > >>>>>>>>>> information, privileged material (including material > >>>> protected > >>>>>> by the > >>>>>>>>>> solicitor-client or other applicable privileges), or > >>>> constitute > >>>>>>>>> non-public > >>>>>>>>>> information. Any use of this information by anyone other > >>>>>>>>>> than > >>>>>> the > >>>>>>>>> intended > >>>>>>>>>> recipient is prohibited. If you have received this > >>>> transmission > >>>>>> in > >>>>>>>> error, > >>>>>>>>>> please immediately reply to the sender and delete this > >>>>>> information from > >>>>>>>>>> your system. Use, dissemination, distribution, or > >>>> reproduction > >>>>>> of this > >>>>>>>>>> transmission by unintended recipients is not authorized > >>>>>>>>>> and > >>>> may > >>>>>> be > >>>>>>>>> unlawful. > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> -- > >>>>>>>>> Gord Tanner > >>>>>>>>> Senior Developer / Code Poet tinyHippos Inc. > >>>>>>>>> @tinyhippos > >>>>>>> > >>>>>>> > >>>>>>> > >>>>>>> -- > >>>>>>> Gord Tanner > >>>>>>> Senior Developer / Code Poet > >>>>>>> tinyHippos Inc. > >>>>>>> @tinyhippos > > >
