Hi Markus, just to explain why "it cannot be used": Imagine: I am a newcomer, and I am developing oc.scheduler.js, and I have oc.upload.new.js already running. I'm looking at oc.upload.new.js, and see function "createSeries". and That's exactly what I need!
But I cannot call this function because It contains a call to: ocUpload.UI.setProgress so it's only for oc.upload.new.js So the easiest way for me is to create my own "createSeries" with call to ocSched.UI.setProgress and it turns that I also have to change the name of createSeries to createSeriesFromSearchText -Pawel --- On Fri, 5/25/12, Markus Moormann <[email protected]> wrote: > From: Markus Moormann <[email protected]> > Subject: Re: [Opencast Matterhorn] duplication of the code... a small step. > To: "Opencast Matterhorn" <[email protected]> > Date: Friday, May 25, 2012, 11:55 PM > Hi Pawel, > > Am 25.05.2012 18:37, schrieb Pawel Fic: > > Functions: > > oc.scheduler.js / createSeriesFromSearchText > > and > > oc.upload.new.js / createSeries > > > > do exactly the same thing. They cannot be used because > of: > > ocUpload.UI.setProgress in > createSeries > what means "they cannot be used"? > otherwise I'm +1, go ahead and please add me for the > review. > > Markus > > > > > > ---- > > Well, my proposal is to get group such functions in: > > oc.common.functions.js > > and put there functions like: > > createSeriesFromSearchText (sheduler > / upload) > > and: > > $('#seriesSelect').autocomplete({ > (oc.scheduler.js) > > > > > initSeriesAutocomplete() (oc.upload.new.js) > > > > > > > > We could use oc.utils for this purpose. > > > > > > So :-) > > What do you think about it ? > > > > > > I am not sure if I can report it as a bug. > > In case something will change in series creation > (mandatory parameter, new rules) - there will be only one > place where it needs to be changed. > > > > > > _______________________________________________ > > Matterhorn mailing list > > [email protected] > > http://lists.opencastproject.org/mailman/listinfo/matterhorn > > > > > > To unsubscribe please email > > [email protected] > > _______________________________________________ > > > > > _______________________________________________ > Matterhorn mailing list > [email protected] > http://lists.opencastproject.org/mailman/listinfo/matterhorn > > > To unsubscribe please email > [email protected] > _______________________________________________ > _______________________________________________ Matterhorn mailing list [email protected] http://lists.opencastproject.org/mailman/listinfo/matterhorn To unsubscribe please email [email protected] _______________________________________________
