I have already attached it in jira. On Aug 7, 2012 3:07 PM, "Maxim Solodovnik" <[email protected]> wrote:
> I need to see your current code :) > please attach it to the issue and I'll review it. > > On Tue, Aug 7, 2012 at 4:28 PM, Ankur Ankan <[email protected]> wrote: > > > Actually I have changed the code after you had reviewed it. > > Now I am saving the values that I am taking from the user in the > > configuaration file in a single line: > > > > this.setuserproperty(server1_baseurl,this.server1_baseurl.getvalue(),1); > > > > And I will have to save the value one by one like this. > > Other than this I am accessing the value just once from the config file. > > So I think there's nowhere I could implement the array. Am I right? > > On Aug 7, 2012 2:36 PM, "Maxim Solodovnik" <[email protected]> wrote: > > > > > You can have an array with server details. > > > This will be scalable solution i.e. you can change the server count to > be > > > 1, 2, ..... any number :) > > > > > > On Tue, Aug 7, 2012 at 3:37 PM, Ankur Ankan <[email protected]> > > wrote: > > > > > > > Hi maxim, > > > > > > > > You had earlier told me to use a for loop as: > > > > > > > > for (var i = 0; i < 5; ++i) { var baseurl_text = new > > > > DwtText({parent:this.pView, name:"server" + (i+1) + "_baseurl_text", > > > > id:"server" + (i+1) + "_baseurl_text"}); baseurl_text.setText("URL"); > > var > > > > baseurl = new DwtInputField ({parent:this.pView, name: "server" + > > (i+1) + > > > > "_baseurl", id: "server" + (i+1) + "_baseurl"}); > > > > > > > > But I am having a doubt about it. Earlier I had done this like > > > > this.server1_baseurl = new DwtInputField({parent....}) > > > > > > > > and was accessing the entered value through > > > > this.server1_baseurl.getvalue(); > > > > > > > > and if I implement a for loop I am unable to figure out how would I > be > > > able > > > > to access the value. > > > > > > > > Need your help over this. > > > > On Aug 7, 2012 1:35 PM, "Ankur Ankan" <[email protected]> wrote: > > > > > > > > > Hi Maxim, > > > > > > > > > > I have removed the unnecessary comments and have created the helper > > > > > function to construct the URL as you had said. > > > > > About using an array and using a for loop after that, I am working > on > > > it > > > > > and will submit the patch tomorrow. > > > > > > > > > > > > > > > On Tue, Aug 7, 2012 at 12:54 PM, Maxim Solodovnik < > > > [email protected] > > > > >wrote: > > > > > > > > > >> Hello Ankur, > > > > >> > > > > >> Were you able to clean up your code based on my last review)? > > > > >> Is the code ready for the next review? > > > > >> > > > > >> On Sun, Aug 5, 2012 at 12:44 PM, Ankur Ankan < > [email protected]> > > > > >> wrote: > > > > >> > > > > >> > Hi, > > > > >> > > > > > >> > I have committed the code to jira and the plugin is working fine > > > but I > > > > >> > think I still need to work on it's looks. > > > > >> > > > > > >> > And the test server is not accessible. Is the server down?? > > > > >> > > > > > >> > On Thu, Aug 2, 2012 at 7:23 PM, [email protected] < > > > > >> > [email protected] > > > > >> > > wrote: > > > > >> > > > > > >> > > Ok! > > > > >> > > > > > > >> > > Eugen would you like to act as Release Manager for the > SugarCRM > > > > plugin > > > > >> > > or do you want me to do it? > > > > >> > > > > > > >> > > Sebastian > > > > >> > > > > > > >> > > 2012/8/2 Eugen Schwert <[email protected]>: > > > > >> > > > @Sebastian the SugarCRM artefacts are okay. > > > > >> > > > > > > > >> > > > 2012/8/2 [email protected] <[email protected]>: > > > > >> > > >> @Ankur That is fine, do you have a demo space available for > > > > >> testing? > > > > >> > > >> @Josh: I will send you some user that has access to the > > > SOAP/REST > > > > >> API > > > > >> > > >> of demo.openmeetings.de so you can test. > > > > >> > > >> For the general "openmeetings_gateway.php" you might simply > > > > compare > > > > >> > > >> the gateway's of all plugins to see which they have in > > common. > > > It > > > > >> > > >> should be almost identical. > > > > >> > > >> > > > > >> > > >> @Eugen: Can you please check if the following artefcts are > > > okay: > > > > >> > > >> > > > > >> > > > > > > >> > > > > > >> > > > > > > > > > > https://builds.apache.org/view/M-R/view/OpenMeetings/job/OpenMeetings%20SugarCRM%20Plugin/ > > > > >> > > >> > > > > >> > > >> and please also complete the Changelog: > > > > >> > > >> > > > > >> > > > > > > >> > > > > > >> > > > > > > > > > > https://svn.apache.org/repos/asf/incubator/openmeetings/trunk/plugins/sugarcrm/6.x.x/CHANGELOG > > > > >> > > >> > > > > >> > > >> Thanks! > > > > >> > > >> Sebastian > > > > >> > > >> > > > > >> > > >> 2012/8/2 Ankur Ankan <[email protected]>: > > > > >> > > >>> I need a two days more. > > > > >> > > >>> On Aug 2, 2012 9:08 AM, "Josh Dolitsky" < > > [email protected]> > > > > >> wrote: > > > > >> > > >>> > > > > >> > > >>>> Also, I've recently been looking at how to create Google > > > Chrome > > > > >> > > plugins, is > > > > >> > > >>>> there perhaps something cool we could do with a plug-in? > > Too > > > > >> much? > > > > >> > > >>>> > > > > >> > > >>>> Josh > > > > >> > > >>>> > > > > >> > > >>>> On Wed, Aug 1, 2012 at 11:35 PM, Josh Dolitsky < > > > > >> [email protected] > > > > >> > > > > > > >> > > >>>> wrote: > > > > >> > > >>>> > > > > >> > > >>>> > @Sebastian I've just started playing with the demo > space, > > > but > > > > >> have > > > > >> > > an > > > > >> > > >>>> > issue (see screenshot). First, should I be using > > > > >> > > >>>> > http://demo.openmeetings.de/openmeetings as the OM > > > > >> installation > > > > >> > > path? > > > > >> > > >>>> > Also, if this is the case, I will need an admin account > > to > > > > >> create > > > > >> > > new > > > > >> > > >>>> > rooms, right? Should I just set to localhost? > > > > >> > > >>>> > > > > > >> > > >>>> > Also, I would be very interested in helping with the > PHP > > > > >> gateway > > > > >> > to > > > > >> > > make > > > > >> > > >>>> > it extensible for future REST integrations. > > > > >> > > >>>> > > > > > >> > > >>>> > Josh > > > > >> > > >>>> > > > > > >> > > >>>> > > > > > >> > > >>>> > On Wed, Aug 1, 2012 at 2:24 PM, [email protected]< > > > > >> > > >>>> > [email protected]> wrote: > > > > >> > > >>>> > > > > > >> > > >>>> >> @Maxim yes that would be great! > > > > >> > > >>>> >> > > > > >> > > >>>> >> Sebastian > > > > >> > > >>>> >> > > > > >> > > >>>> >> 2012/8/1 Maxim Solodovnik <[email protected]>: > > > > >> > > >>>> >> > I'll create build file with rat task. > > > > >> > > >>>> >> > Additionally I think it would be good to have > > > > >> > > openmeetings_gateway as > > > > >> > > >>>> >> > separate project shared with all php projects (as > > future > > > > >> task > > > > >> > > :)) ) > > > > >> > > >>>> >> > > > > > >> > > >>>> >> > On Wed, Aug 1, 2012 at 9:00 PM, > [email protected]< > > > > >> > > >>>> >> [email protected] > > > > >> > > >>>> >> >> wrote: > > > > >> > > >>>> >> > > > > > >> > > >>>> >> >> Hi Maxim, > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> which packages need to be created from the source > to > > > > >> create a > > > > >> > > Joomla > > > > >> > > >>>> >> >> Plugin? > > > > >> > > >>>> >> >> Is it only one component and one module or are > there > > > more > > > > >> and > > > > >> > > what > > > > >> > > >>>> >> >> naming conventions need to be applied? > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> I've created a build script for the Moodle plugin: > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> > > > > >> > > >>>> > > > > >> > > > > > > >> > > > > > >> > > > > > > > > > > https://svn.apache.org/repos/asf/incubator/openmeetings/trunk/plugins/moodle_plugin/build.xml > > > > >> > > >>>> >> >> It should be easy to recfator the build script to > > > produce > > > > >> > > similar > > > > >> > > >>>> >> >> packages for the Joomla Plugin. > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> Thanks! > > > > >> > > >>>> >> >> Sebastian > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> 2012/8/1 Maxim Solodovnik <[email protected]>: > > > > >> > > >>>> >> >> > Joomla plugin need to be tested. > > > > >> > > >>>> >> >> > I was hoping on users feedback. > > > > >> > > >>>> >> >> > The code should be clean. All files should have > > > license > > > > >> > > headers. > > > > >> > > >>>> >> Need to > > > > >> > > >>>> >> >> > run rat on it > > > > >> > > >>>> >> >> > On Aug 1, 2012 6:23 PM, "[email protected]" > < > > > > >> > > >>>> >> [email protected]> > > > > >> > > >>>> >> >> > wrote: > > > > >> > > >>>> >> >> > > > > > >> > > >>>> >> >> >> I have configured and update the Moodle plugin > on > > > > >> Jenkins: > > > > >> > > >>>> >> >> >> > > > > >> > > https://builds.apache.org/job/OpenMeetings%20Moodle%20Plugin/ > > > > >> > > >>>> >> >> >> > > > > >> > > >>>> >> >> >> @Maxim: What is the status of the Joomla plugin, > > is > > > it > > > > >> > ready > > > > >> > > to > > > > >> > > >>>> >> release? > > > > >> > > >>>> >> >> >> @Eugen: What is the status of the SugarCRM > plugin, > > > is > > > > it > > > > >> > > ready to > > > > >> > > >>>> >> >> release? > > > > >> > > >>>> >> >> >> @German/Alexei: Teambox/Jitsi should be released > > as > > > > >> plugin? > > > > >> > > >>>> >> >> >> @Josh: ATutor? > > > > >> > > >>>> >> >> >> @Ankur: Zimbra? > > > > >> > > >>>> >> >> >> > > > > >> > > >>>> >> >> >> Thanks! > > > > >> > > >>>> >> >> >> Sebastian > > > > >> > > >>>> >> >> >> -- > > > > >> > > >>>> >> >> >> Sebastian Wagner > > > > >> > > >>>> >> >> >> https://twitter.com/#!/dead_lock > > > > >> > > >>>> >> >> >> http://www.openmeetings.de > > > > >> > > >>>> >> >> >> http://www.webbase-design.de > > > > >> > > >>>> >> >> >> http://www.wagner-sebastian.com > > > > >> > > >>>> >> >> >> [email protected] > > > > >> > > >>>> >> >> >> > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> >> -- > > > > >> > > >>>> >> >> Sebastian Wagner > > > > >> > > >>>> >> >> https://twitter.com/#!/dead_lock > > > > >> > > >>>> >> >> http://www.openmeetings.de > > > > >> > > >>>> >> >> http://www.webbase-design.de > > > > >> > > >>>> >> >> http://www.wagner-sebastian.com > > > > >> > > >>>> >> >> [email protected] > > > > >> > > >>>> >> >> > > > > >> > > >>>> >> > > > > > >> > > >>>> >> > > > > > >> > > >>>> >> > > > > > >> > > >>>> >> > -- > > > > >> > > >>>> >> > WBR > > > > >> > > >>>> >> > Maxim aka solomax > > > > >> > > >>>> >> > > > > >> > > >>>> >> > > > > >> > > >>>> >> > > > > >> > > >>>> >> -- > > > > >> > > >>>> >> Sebastian Wagner > > > > >> > > >>>> >> https://twitter.com/#!/dead_lock > > > > >> > > >>>> >> http://www.openmeetings.de > > > > >> > > >>>> >> http://www.webbase-design.de > > > > >> > > >>>> >> http://www.wagner-sebastian.com > > > > >> > > >>>> >> [email protected] > > > > >> > > >>>> >> > > > > >> > > >>>> > > > > > >> > > >>>> > > > > > >> > > >>>> > > > > >> > > >> > > > > >> > > >> > > > > >> > > >> > > > > >> > > >> -- > > > > >> > > >> Sebastian Wagner > > > > >> > > >> https://twitter.com/#!/dead_lock > > > > >> > > >> http://www.openmeetings.de > > > > >> > > >> http://www.webbase-design.de > > > > >> > > >> http://www.wagner-sebastian.com > > > > >> > > >> [email protected] > > > > >> > > > > > > >> > > > > > > >> > > > > > > >> > > -- > > > > >> > > Sebastian Wagner > > > > >> > > https://twitter.com/#!/dead_lock > > > > >> > > http://www.openmeetings.de > > > > >> > > http://www.webbase-design.de > > > > >> > > http://www.wagner-sebastian.com > > > > >> > > [email protected] > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > >> > > > > >> -- > > > > >> WBR > > > > >> Maxim aka solomax > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > WBR > > > Maxim aka solomax > > > > > > > > > -- > WBR > Maxim aka solomax >
