+1 to Jesse's suggestion of Party. And the events thing.
On Thu, Nov 29, 2012 at 10:51 PM, Shazron <[email protected]> wrote: > Ok this is the essence of what the previous threads I linked to guys :) > > > On Thu, Nov 29, 2012 at 1:43 PM, Jesse <[email protected]> wrote: > >> Have the AppDelegate raise events when things like >> 'didReceiveLocalNotification' happen. >> Have plugins subscribe to the event. >> Party. >> >> >> On Thu, Nov 29, 2012 at 1:30 PM, Anis KADRI <[email protected]> wrote: >> > It's tricky exactly because of that reason (i.e multiple plugins editing >> > the same files/methods). I don't see a clean solution to this problem. >> One >> > way would be to just apply the patch. If it fails, notify the user and >> > prompt them to apply the code manually. Bright minds out there, your >> > thoughts are welcome. >> > >> > >> > >> > >> > On Thu, Nov 29, 2012 at 12:39 PM, Filip Maj <[email protected]> wrote: >> > >> >> I think I had discussions with someone about this particular feature. >> >> >> >> What about having empty stubs for these functions that some plugins need >> >> to hook into (I.e. didReceiveLocalNotification) and then have the >> tooling >> >> drop code in where necessary for these functions? This can be a slippery >> >> slope (multiple plugins battling over a single function) but not worth a >> >> shot? >> >> >> >> On 11/28/12 11:13 PM, "Anis KADRI" <[email protected]> wrote: >> >> >> >> >On Wed, Nov 28, 2012 at 10:43 PM, Brian LeRoux <[email protected]> wrote: >> >> > >> >> >> would this not be an indication that we need to modify the plugin >> api? >> >> >> (code munging would be a bad code smell for a pluggable architecture >> >> >> imo) >> >> >> >> >> > >> >> >Yes. The current specification does not support this use case. >> >> >I can't seem to find a clean/elegant solution to the problem. diff'ing >> >> >does >> >> >not work well if there are multiple plugins appending code to existing >> >> >files. One option would just be to prompt the user to append the code >> >> >manually (but that would not be too cool with cordova-client). >> >> > >> >> >Shaz, I believe I understand that we can expand AppDelegate to support >> >> >even >> >> >more overrides (including the didReceiveLocalNotification) but I >> believe >> >> >the real issue here is appending code in general and this issue is not >> >> >limited to iOS. We can keep this on the backburner for now as it is not >> >> >required by most plugins. >> >> >> >> >> >> >> >> -- >> @purplecabbage >> risingj.com >>
