March 2009 - xwiki-devs@xwiki.org

[xwiki-devs] Office Importer - Importing into multiple wiki pages
by Asiri Rathnayake 11 Mar '09

11 Mar '09

Hi devs, To implement the above functionality I have created the following UI: http://i43.tinypic.com/28l7x2u.png which was dervied from the mockups located at http://incubator.myxwiki.org/xwiki/bin/view/Mockups/ImportCompositeDocument Descriptions of various fields are as follows: * Document - The office document to be uploaded (and imported) * Style filtering - Whether to filter office styles or not * Heading level to split - If the user wishes to split the imported document into multiple wiki pages, he has to select the heading level (h1, h2, h3... h6) to be used when splitting the document. If the user does not select a heading level, the document will be imported as it is (no splitting). * Custom split regex - If the user wants to further refine the split criterion (based on the content of header) this field allows him to specify that criterion through a regular expression. Example regular expression: <b>Section<b>.* Open Question: Aren't regular expressions bit too technical for users? * Target space - This is where the resulting document(s) will land. * Target (master) page - The main document holding the TOC (in case of splitting), otherwise this is the name of resulting wiki page. * Child pages naming method - If the document is split into multiple pages, pages should be named according to some criterion. This combo box allows users to specify that criterion. Regarding the implementation, we have two possible approaches. 1. Implement the splitting in w3c dom level (xhtml) 2. Implement the splitting in XDOM level * In the first approach we will navigate through the child elements directly under <body> tag and find matching heading elements. For the regex, we will have to serialize the heading element so that the regex can be evaluated. Heading elements can be serialized as explained here: http://forums.sun.com/thread.jspa?threadID=698475 * In the second approach we can either use XDOM operations or use a SplittingChainingListener. But I don't know whether regex matching is possible with this scheme. Also, regardless of the method we follow, there will be a problem with large office documents (say 100MB or so). Loading such a file into memory (dom or xdom) would not be a good idea. I haven't decided which method to go with yet. So it will be really great if we can sort this out as soon as possible. Thanks. - Asiri

4 4

Re: [xwiki-devs] [xwiki-users] SMTP authentications
by Lilianne E. Blaze 11 Mar '09

11 Mar '09

Norguir Bellagio wrote: > Hi, > > I have crawled the mailing lists and support forums on this but I am > unable to figure out the answer. > > I am evaluating Xwiki and love Workspaces from what I've seen so far. > The one thing that really bogs me down is the inability to register an > authenticated SMTP server. I have read there is a patch from Liliane, > and the feature is included on 1.6 but I just can't seem to understand > how to get is to work. In an evaluation perspective (and very much so as > well in production), not being able to use authenticated SMTP server is > a no go for me. > Could you please point me to a resource that clarifies what the > situation is and possibly, the steps to enable it on 1.7. > > Thanks ! > > norguir Actually I'm still waiting for it :/ I just made a quick check and it looks like that patch was not committed, there's no support for setting username and password in Administration, and registration mails still go via deprecated XWiki.sendMessage, which is using Apache Commons SMTPClient (made obsolete by JavaMail ages ago) and has no support for username/password and extra properties. As for mail.smtp.starttls.enable=true, AFAIK when used alone it simply does not make sense. _Please_ do something about it :/ If I remember correctly last time the main issue was that the code had to be integrated in core instead of mail plugin, please, can't we have the thing working at all first, worry about module separation later? Basic mail support _is_ core functionality. For me it's a showstopper, and I don't have enough time to maintain a private fork, either smtp auth goes in, or I'm going out :/ And before someone says it's open source and I should stop bitching and submit a patch - I _did_ submit a patch months ago :/ Greetings, Lilianne

3 3

Re: [xwiki-devs] [xwiki-notifications] r17373 - in platform: core/trunk/xwiki-core/src/main/resources core/trunk/xwiki-officeimporter core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter core/trunk/xwiki-officeimporter/src/main/java
by Asiri Rathnayake 09 Mar '09

09 Mar '09

Hi Sergiu, > Since you only committed in the trunk, should this be @since 1.9M1? The > same for all the other @since tags. Merged with 1.8 branch today. > This is good for a standalone Java program. When it comes to webapps, > the lifecycle is much more complex, since a webapp can be stopped and > reloaded at will, in a clustered environment it can be moved from one > computer to another, or it can even be stopped from within the > application itself. This code is good for the moment, but it will have > to be changed as soon as we introduce lifecycle events in the > observation component. > I agree :) Creating the bridge for each new context is a waste of CPU and RAM. > Since the bridge is stateless, it can be a member initialized in the > constructor. > > > + context.put(VELOCITY_CONTEXT_KEY, new > OpenOfficeServerManagerVelocityBridge(oomanager, docBridge)); > > + } > > +} > > > Fixed. > Why not override toString()? > > > + public String getDescription() > > + { > > + return this.stateDescription; > > + } > > + } > Fixed. > > in in > > > + * These configuration properties are defined in in XWiki's global > configuration file using the prefix of > > + * "officeimporter". > > + * </p> > > + * > > + * @version $Id$ > > + * @since 1.8RC3 > > + */ > > +public interface OpenOfficeServerConfiguration > Fixed. > I'm not sure that checking for specific values is a good decision. > Perhaps checking for non-null, non-empty and non-negative? > > > + > assertEquals(OfficeUtils.getDefaultOfficeHome().getAbsolutePath(), > configuration.getHomePath()); > > + > assertEquals(OfficeUtils.getDefaultProfileDir().getAbsolutePath(), > configuration.getProfilePath()); > > + assertEquals(50, configuration.getMaxTasksPerProcess()); > > + assertEquals(30000, configuration.getTaskExecutionTimeout()); > > + } > > +} > Fixed. > Is this attachment used anywhere? > > > +<attachment> > > +<filename>icon.png</filename> > > +<filesize>8674</filesize> > > +<author>XWiki.Admin</author> > > +<date>1236277246000</date> > > +<version>1.0</version> > > +<comment></comment> > > +</attachment> > Yes, it's used as the openoffice server administration icon in XWikiPreferences page. This icon will be replaced by a better one (I have asked laurent). > I usually delete the tag objects from the XML file before committing. > Fixed (Removed tag objects) > > +#if($hasAdmin) > > + #info($msgConfiguration) > > + #if($request.action && $request.action=="stop") > > This is best done like this: > #if("$!request.action" == 'stop') > > > + #if(!$oomanager.stopServer()) > > + #error($oomanager.lastErrorMessage) > > + #end > > + #elseif($request.action && $request.action=="start") > > + #if(!$oomanager.startServer()) > > + #error($oomanager.lastErrorMessage) > > + #end > > + #elseif($request.action && $request.action=="restart") > > Since you're using it several times, you could store the action in a > variable at the top: > #set($currentAction = "$!request.action") > Fixed (learnt about $! just now) > What's with the paragraphs? IMO they aren't needed, just write: > > $msgServerPath | $serverPath > > > +<p>$msgServerPath</p>|<p>$serverPath</p> > > +<p>$msgServerProfile</p>|<p>$serverProfile</p> > > +<p>$msgServerState</p>|<p>$currentState</p> > Well, without the paragraphs the table looks bit too tight. > > Better: > > Setting for the OpenOffice instance controlled by the office importer > plugin. > > > +# OpenOffice > > > +#------------------------------------------------------------------------------------- > > + > Fixed. > > Since 1.9M1? > > > +#-# [Since 1.8RC3] > > +#-# Path to openoffice installation. > > +# openoffice.homePath=/opt/openoffice.org3/ > > + > > +#-# [Since 1.8RC3] > > +#-# Path to openoffice execution profile. > > Is this right? For me it's /home/user/.ooo3/ > > > +# openoffice.profilePath=/home/user/.openoffice.org/3 > This is the default path structure returned by jodconverter. I think this is valid for most Linux systems. Thanks. - Asiri

1 0

[xwiki-devs] [VOTE] Move components out of the internal package for the rendering module
by Vincent Massol 09 Mar '09

09 Mar '09

Hi, Some time ago we agreed that component implementations should be located in the internal package. I now believe it's wrong if we want to allow users to use our modules separately from the rest of XWiki as standard java beans. I think it's a good property to have that users can integrate our modules into their own code without taking the whole of XWiki. This is what I'm trying to do with the rendering module. Thus I'm proposing to move the components out of the internal module *only* for the rendering module for now even though I suspect that we'll want to do that for all modules eventually. Here's my +1 Thanks -Vincent

3 6

[xwiki-devs] [VOTE] Make spaces non meaningfull in XWiki 2.0 syntax (was [Rendering] Should we move to non meaningful spaces?)
by Thomas Mortagne 09 Mar '09

09 Mar '09

Hi devs, We have to make a decision about that. So here are the proposals: 1) remove the block leading and trainling spaces * The main goal is to make source formatting for tables for example more readable 2) make the spaces inside paragraph non meaningfull * Meaning an HTML like behavior where multiple spaces give one space 3) in case of 1) or 2) use ~<space> as non breaking space WDYT ? +0,5 for 1) it's not critical for me but i'm not against it and we already decided to remove space before list item, headers etc. -0 for 2) I don't see the need for that and it's a lot easier for the parser to make spaces meaningfull (what to do when you have "test ** bold**" and things like that) +1 for 3) On Sat, Feb 28, 2009 at 15:44, Vincent Massol <vincent(a)massol.net> wrote: > Hi, > > This is our last chance to change this behavior. We've found several > places where having meaningful spaces are counter-productive: > > * in table cells since we can't align table anymore. For example: > > |= column1 |= column2 > | this is some para | second column > | hello | world > > (not sure this will be rendered nicely in mail but you see what I mean) > > * in scripts since having meaningful spaces prevents us from aligning > velocity or groovy scripts. For ex we can't write: > > #if (....) > #if (...) > do something > # end > #end > > To see a better example have a look at http://tinyurl.com/ahz669 > > What I think users real want are meaningful new lines but I see cons > overweighting pros for having meaningful white spaces. Thus I'm think > we should strip whitespaces at beginning and end of lines including > for line breaks. > I'm slightly less sure for multiple spaces between words but even > there I think we could strip them have users use {{{ }}} to put a non > breaking space for ex (or introduce a {{space/}} macro or another > special syntax although I'd rather we don't introduce a new syntax). > > WDYT? > > Thanks > -Vincent > http://xwiki.com > http://xwiki.org > http://massol.net > > > > > > > _______________________________________________ > devs mailing list > devs(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/devs > -- Thomas Mortagne

7 22

[xwiki-devs] XE-391: fencepost error that occurs when there's only one document revision
by Niels Mayer 08 Mar '09

08 Mar '09

http://jira.xwiki.org/jira/browse/XE-391 I believe this issue occurs when there is only one initial revision to a > document, and 'history' is selected and attempts to diff between version 1 > (which exists) and version 2 (which doesn't). > > The code needs to be special-cased for this > http://en.wikipedia.org/wiki/Off-by-one_error#Fencepost_error > > Here's some recent examples, caused by search engines: > > 2009-03-03 14:44:16,472 [/xwiki/bin/view/XWiki/SchedulerJobClass?viewer=changes&rev1=$%7Brev1%7D&rev2=$%7Brev2%7D] [TP-Processor35] ERROR api.XWiki - Failed to read versioned document > org.suigeneris.jrcs.rcs.InvalidVersionNumberException: ${rev1} > at org.suigeneris.jrcs.rcs.Version.<init>(Version.java:120) > > What's with scheduler job class? one revision: > > History of SchedulerJobClass > Version Editor Date Comment > 1.1 NielsMayer.com Administrator Dec 15, 2006 03:45:59 GMT-08:00 > > Another: > > 2009-03-07 03:24:39,164 /xwiki/bin/view/XWiki/WebPreferences?viewer=changes&rev1=$%7Brev1%7D&rev2=$%7Brev2%7D] [TP-Processor6] ERROR api.XWiki - Failed to read versioned document > org.suigeneris.jrcs.rcs.InvalidVersionNumberException: ${rev2} > at org.suigeneris.jrcs.rcs.Version.<init>(Version.java:120) > > What's with WebPreferences? One revision: > > Version Editor Date Comment > 1.1 NielsMayer.com Administrator Apr 20, 2007 09:17:52 GMT-08:00 > > [ Show » <http://jira.xwiki.org/jira/browse/XE-391> ] Niels Mayer, http://nielsmayer.com<http://jira.xwiki.org/jira/secure/ViewProfile.jspa?name=nielsmayer>added a comment - 08/Mar/09 19:04 - edited I believe this issue occurs when there is only one initial revision to a document, and 'history' is selected and attempts to diff between version 1 (which exists) and version 2 (which doesn't). The code needs to be special-cased for this http://en.wikipedia.org/wiki/Off-by-one_error#Fencepost_error Here's some recent examples, caused by search engines: 2009-03-03 14:44:16,472 [/xwiki/bin/view/XWiki/SchedulerJobClass?viewer=changes&rev1=$%7Brev1%7D&rev2=$%7Brev2%7D] [TP-Processor35] ERROR api.XWiki - Failed to read versioned document org.suigeneris.jrcs.rcs.InvalidVersionNumberException: ${rev1} at org.suigeneris.jrcs.rcs.Version.<init>(Version.java:120) What's with scheduler job class? one revision: History of SchedulerJobClass Version Editor Date Comment 1.1 NielsMayer.com Administrator Dec 15, 2006 03:45:59 GMT-08:00 Another: 2009-03-07 03:24:39,164 /xwiki/bin/view/XWiki/WebPreferences?viewer=changes&rev1=$%7Brev1%7D&rev2=$%7Brev2%7D] [TP-Processor6] ERROR api.XWiki - Failed to read versioned document org.suigeneris.jrcs.rcs.InvalidVersionNumberException: ${rev2} at org.suigeneris.jrcs.rcs.Version.<init>(Version.java:120) What's with WebPreferences? One revision: Version Editor Date Comment 1.1 NielsMayer.com Administrator Apr 20, 2007 09:17:52 GMT-08:00 Niels http://nielsmayer.com

1 0

Re: [xwiki-devs] [xwiki-notifications] r17373 - in platform: core/trunk/xwiki-core/src/main/resources core/trunk/xwiki-officeimporter core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/configuration core/trunk/xwiki-officeimporter
by Sergiu Dumitriu 08 Mar '09

08 Mar '09

asiri (SVN) wrote: > Author: asiri > Date: 2009-03-06 10:44:22 +0100 (Fri, 06 Mar 2009) > New Revision: 17373 > > Added: > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/ > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManager.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityBridge.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityContextInitializer.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/configuration/ > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeServerConfiguration.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/ > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManager.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManagerException.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/configuration/ > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/configuration/OpenOfficeServerConfiguration.java > platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/ > platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManagerTest.java > platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/configuration/ > platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeConfigurationTest.java > platform/xwiki-applications/trunk/officeimporter/src/main/resources/XWiki/OfficeImporterAdmin.xml > Modified: > platform/core/trunk/xwiki-core/src/main/resources/ApplicationResources.properties > platform/core/trunk/xwiki-officeimporter/pom.xml > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/OfficeImporterContext.java > platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/transformer/OfficeToHtmlTransformer.java > platform/core/trunk/xwiki-officeimporter/src/main/resources/META-INF/plexus/components.xml > platform/xwiki-applications/trunk/administration/src/main/resources/XWiki/AdminSheet.xml > platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm > Log: > XWIKI-3300: Add support for controlling the openoffice server process from XWiki UI > > * Implemented. > > Modified: platform/core/trunk/xwiki-core/src/main/resources/ApplicationResources.properties > =================================================================== > --- platform/core/trunk/xwiki-core/src/main/resources/ApplicationResources.properties 2009-03-05 18:26:09 UTC (rev 17372) > +++ platform/core/trunk/xwiki-core/src/main/resources/ApplicationResources.properties 2009-03-06 09:44:22 UTC (rev 17373) > @@ -1307,6 +1307,15 @@ > xe.officeimporter.results.missingpage=Missing target page name. Please {0} and correct it. > xe.officeimporter.results.result=result > xe.officeimporter.results.success=Conversion succeeded. You can view the {0}, or you can {1} to convert another document. > +xe.officeimporter.openoffice.serverpath=Server Path > +xe.officeimporter.openoffice.serverprofile=Server Profile > +xe.officeimporter.openoffice.configuration=You can override *{0}* and *{1}* variables through *WEB-INF/xwiki.properties* file. > +xe.officeimporter.openoffice.serverstate=Server State > +xe.officeimporter.openoffice.actions=Actions > +xe.officeimporter.openoffice.actions.start=Start > +xe.officeimporter.openoffice.actions.stop=Stop > +xe.officeimporter.openoffice.actions.restart=Restart > +xe.officeimporter.openoffice.update=Update > > # Panels application > xe.panels.classedit.youare=You are editing > > Modified: platform/core/trunk/xwiki-officeimporter/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/pom.xml 2009-03-05 18:26:09 UTC (rev 17372) > +++ platform/core/trunk/xwiki-officeimporter/pom.xml 2009-03-06 09:44:22 UTC (rev 17373) > @@ -50,6 +50,11 @@ > </dependency> > <dependency> > <groupId>org.xwiki.platform</groupId> > + <artifactId>xwiki-core-configuration</artifactId> > + <version>${pom.version}</version> > + </dependency> > + <dependency> > + <groupId>org.xwiki.platform</groupId> > <artifactId>xwiki-core-velocity</artifactId> > <version>${pom.version}</version> > </dependency> > @@ -59,28 +64,16 @@ > <version>${pom.version}</version> > </dependency> > <dependency> > - <groupId>com.artofsolving</groupId> > + <groupId>net.sf.jodconverter</groupId> > <artifactId>jodconverter</artifactId> > - <version>2.2.1</version> > + <version>3.0-beta-1</version> > </dependency> > -  > <dependency> > - <groupId>org.slf4j</groupId> > - <artifactId>slf4j-log4j12</artifactId> > - <version>1.4.3</version> > - </dependency> > - <dependency> > <groupId>org.xwiki.platform</groupId> > <artifactId>xwiki-core-rendering-parser-wikimodel</artifactId> > <version>${pom.version}</version> > <scope>runtime</scope> > </dependency> > - <dependency> > - <groupId>org.xwiki.platform</groupId> > - <artifactId>xwiki-core-configuration</artifactId> > - <version>${pom.version}</version> > - <scope>runtime</scope> > - </dependency> >  > <dependency> > <groupId>com.xpn.xwiki.platform.tools</groupId> > > Modified: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/OfficeImporterContext.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/OfficeImporterContext.java 2009-03-05 18:26:09 UTC (rev 17372) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/OfficeImporterContext.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -24,11 +24,12 @@ > import java.util.List; > import java.util.Map; > > +import net.sf.jodconverter.DefaultDocumentFormatRegistry; > +import net.sf.jodconverter.DocumentFormat; > +import net.sf.jodconverter.DocumentFormatRegistry; > + > import org.xwiki.bridge.DocumentAccessBridge; > > -import com.artofsolving.jodconverter.DefaultDocumentFormatRegistry; > -import com.artofsolving.jodconverter.DocumentFormat; > -import com.artofsolving.jodconverter.DocumentFormatRegistry; > > /** > * Contains all the context information for a particular transformation. While an office document is being transformed > @@ -107,7 +108,7 @@ > DocumentFormatRegistry formatRegistry = new DefaultDocumentFormatRegistry(); > int dot = sourceFileName.lastIndexOf('.'); > if (dot != -1) { > - this.sourceFormat = formatRegistry.getFormatByFileExtension(sourceFileName.substring(dot + 1)); > + this.sourceFormat = formatRegistry.getFormatByExtension(sourceFileName.substring(dot + 1)); > } > if (sourceFormat == null) { > throw new OfficeImporterException("Unable to determine input file format."); > @@ -182,7 +183,7 @@ > */ > public boolean isPresentation() > { > - return PRESENTATION_FORMAT_EXTENSIONS.contains(sourceFormat.getFileExtension().toLowerCase()); > + return PRESENTATION_FORMAT_EXTENSIONS.contains(sourceFormat.getExtension().toLowerCase()); > } > > /** > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManager.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManager.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManager.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,193 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.internal.openoffice; > + > +import java.io.File; > + > +import net.sf.jodconverter.OfficeDocumentConverter; > +import net.sf.jodconverter.office.ManagedProcessOfficeManager; > +import net.sf.jodconverter.office.OfficeException; > +import net.sf.jodconverter.office.OfficeManager; > + > +import org.xwiki.component.logging.AbstractLogEnabled; > +import org.xwiki.component.phase.Initializable; > +import org.xwiki.component.phase.InitializationException; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManager; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManagerException; > +import org.xwiki.officeimporter.openoffice.configuration.OpenOfficeServerConfiguration; > + > +/** > + * Default implementation of {@link OpenOfficeServerManager} component. > + * > + * @version $Id$ Since you only committed in the trunk, should this be @since 1.9M1? The same for all the other @since tags. > + * @since 1.8RC3 > + */ > +public class DefaultOpenOfficeServerManager extends AbstractLogEnabled implements OpenOfficeServerManager, > + Initializable > +{ > + /** > + * The {@link OpenOfficeServerConfiguration} component. > + */ > + private OpenOfficeServerConfiguration configuration; > + > + /** > + * Current oo server process state. > + */ > + private ServerState currentState; > + > + /** > + * The {@link OfficeManager} used to control the openoffice server instance. > + */ > + private OfficeManager officeManager; > + > + /** > + * Flag indicating whether the officeManager is initialized or not. > + */ > + private boolean officeManagerInitialized; > + > + /** > + * The {@link OfficeDocumentConverter} used to convert office documents. > + */ > + private OfficeDocumentConverter documentConverter; > + > + /** > + * {@inheritDoc} > + */ > + public void initialize() throws InitializationException > + { > + currentState = ServerState.STOPPED; > + // Make sure there is no openoffice process left when XE shuts down. This is good for a standalone Java program. When it comes to webapps, the lifecycle is much more complex, since a webapp can be stopped and reloaded at will, in a clustered environment it can be moved from one computer to another, or it can even be stopped from within the application itself. This code is good for the moment, but it will have to be changed as soon as we introduce lifecycle events in the observation component. > + Runtime.getRuntime().addShutdownHook(new Thread() > + { > + public void run() > + { > + try { > + stopServer(); > + } catch (OpenOfficeServerManagerException ex) { > + // Nothing to do. > + } > + } > + }); > + } > + > + /** > + * Initializes the internal {@link OfficeManager}. > + */ > + private void initializeOfficeManager() throws OpenOfficeServerManagerException > + { > + File officeHome = new File(getOfficeHome()); > + File officeProfile = new File(getOfficeProfile()); > + try { > + ManagedProcessOfficeManager managedProcessOfficeManager = > + new ManagedProcessOfficeManager(officeHome, officeProfile); > + managedProcessOfficeManager.setMaxTasksPerProcess(configuration.getMaxTasksPerProcess()); > + managedProcessOfficeManager.setTaskExecutionTimeout(configuration.getTaskExecutionTimeout()); > + this.officeManager = managedProcessOfficeManager; > + this.documentConverter = new OfficeDocumentConverter(officeManager); > + setOfficeManagerInitialized(true); > + } catch (IllegalArgumentException ex) { > + currentState = ServerState.ERROR; > + throw new OpenOfficeServerManagerException("Error while initializing OpenOffice server.", ex); > + } > + } > + > + /** > + * @return whether the officeManager is initialized or not. > + */ > + private boolean isOfficeManagerInitialized() > + { > + return officeManagerInitialized; > + } > + > + /** > + * @param initialized state of the officeManager to be set. > + */ > + private void setOfficeManagerInitialized(boolean initialized) > + { > + this.officeManagerInitialized = initialized; > + } > + > + /** > + * {@inheritDoc} > + */ > + public String getOfficeHome() > + { > + return configuration.getHomePath(); > + } > + > + /** > + * {@inheritDoc} > + */ > + public String getOfficeProfile() > + { > + return configuration.getProfilePath(); > + } > + > + /** > + * {@inheritDoc} > + */ > + public ServerState getServerState() > + { > + return currentState; > + } > + > + /** > + * {@inheritDoc} > + */ > + public void startServer() throws OpenOfficeServerManagerException > + { > + if (ServerState.STOPPED == currentState) { > + if (!isOfficeManagerInitialized()) { > + initializeOfficeManager(); > + } > + try { > + officeManager.start(); > + currentState = ServerState.RUNNING; > + } catch (OfficeException ex) { > + throw new OpenOfficeServerManagerException("Error while starting OpenOffice server.", ex); > + } > + } > + } > + > + /** > + * {@inheritDoc} > + */ > + public void stopServer() throws OpenOfficeServerManagerException > + { > + if (ServerState.RUNNING == currentState) { > + try { > + officeManager.stop(); > + currentState = ServerState.STOPPED; > + } catch (OfficeException ex) { > + throw new OpenOfficeServerManagerException("Error while shutting down OpenOffice server.", ex); > + } finally { > + setOfficeManagerInitialized(false); > + } > + } > + } > + > + /** > + * {@inheritDoc} > + */ > + public OfficeDocumentConverter getDocumentConverter() > + { > + return this.documentConverter; > + } > +} > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityBridge.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityBridge.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityBridge.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,132 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.internal.openoffice; > + > +import org.xwiki.bridge.DocumentAccessBridge; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManager; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManagerException; > + > +/** > + * A bridge between {@link OpenOfficeServerManager} and velocity scripts. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public class OpenOfficeServerManagerVelocityBridge > +{ > + /** > + * The {@link OpenOfficeServerManager} component. > + */ > + private OpenOfficeServerManager oomanager; > + > + /** > + * The {@link DocumentAccessBridge} component. > + */ > + private DocumentAccessBridge docBridge; > + > + /** > + * Holds any error messages thrown during operations. > + */ > + private String lastErrorMessage; > + > + /** > + * Creates a new {@link OpenOfficeServerManagerVelocityBridge} with the provided {@link OpenOfficeServerManager} > + * component. > + */ > + public OpenOfficeServerManagerVelocityBridge(OpenOfficeServerManager oomanager, DocumentAccessBridge docBridge) > + { > + this.oomanager = oomanager; > + this.docBridge = docBridge; > + } > + > + /** > + * Tries to start the oo server process. > + * > + * @return true if the operation succeeds, false otherwise. > + */ > + public boolean startServer() > + { > + boolean success = false; > + if (docBridge.hasProgrammingRights()) { > + try { > + oomanager.startServer(); > + success = true; > + } catch (OpenOfficeServerManagerException ex) { > + this.lastErrorMessage = ex.getMessage(); > + } > + } else { > + this.lastErrorMessage = "Inadequate privileges."; > + } > + return success; > + } > + > + /** > + * Tries to stop the oo server process. > + * > + * @return true if the operation succeeds, false otherwise. > + */ > + public boolean stopServer() > + { > + boolean success = false; > + if (docBridge.hasProgrammingRights()) { > + try { > + oomanager.stopServer(); > + success = true; > + } catch (OpenOfficeServerManagerException ex) { > + this.lastErrorMessage = ex.getMessage(); > + } > + } else { > + this.lastErrorMessage = "Inadequate privileges."; > + } > + return success; > + } > + > + /** > + * @return path to openoffice server installation. > + */ > + public String getOfficeHome() > + { > + return oomanager.getOfficeHome(); > + } > + > + /** > + * @return path to openoffice execution profile. > + */ > + public String getOfficeProfile() > + { > + return oomanager.getOfficeProfile(); > + } > + > + /** > + * @return current status of the oo server process as a string. > + */ > + public String getServerState() > + { > + return oomanager.getServerState().getDescription(); > + } > + > + /** > + * @return any error messages encountered. > + */ > + public String getLastErrorMessage() > + { > + return this.lastErrorMessage; > + } > +} > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityContextInitializer.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityContextInitializer.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/OpenOfficeServerManagerVelocityContextInitializer.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,57 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.internal.openoffice; > + > +import org.apache.velocity.VelocityContext; > +import org.xwiki.bridge.DocumentAccessBridge; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManager; > +import org.xwiki.velocity.VelocityContextInitializer; > + > +/** > + * Puts a reference to {@link OpenOfficeServerManager} in newly created velocity contexts. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public class OpenOfficeServerManagerVelocityContextInitializer implements VelocityContextInitializer > +{ > + /** > + * The key to use for openoffice server manager in the velocity context. > + */ > + public static final String VELOCITY_CONTEXT_KEY = "oomanager"; > + > + /** > + * The {@link OpenOfficeServerManager} component. > + */ > + private OpenOfficeServerManager oomanager; > + > + /** > + * The {@link DocumentAccessBridge} component. > + */ > + private DocumentAccessBridge docBridge; > + > + /** > + * {@inheritDoc} > + */ > + public void initialize(VelocityContext context) > + { Creating the bridge for each new context is a waste of CPU and RAM. Since the bridge is stateless, it can be a member initialized in the constructor. > + context.put(VELOCITY_CONTEXT_KEY, new OpenOfficeServerManagerVelocityBridge(oomanager, docBridge)); > + } > +} > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeServerConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeServerConfiguration.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeServerConfiguration.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,141 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.internal.openoffice.configuration; > + > +import net.sf.jodconverter.office.OfficeUtils; > + > +import org.xwiki.component.phase.Initializable; > +import org.xwiki.component.phase.InitializationException; > +import org.xwiki.configuration.ConfigurationManager; > +import org.xwiki.configuration.ConfigurationSourceCollection; > +import org.xwiki.officeimporter.openoffice.configuration.OpenOfficeServerConfiguration; > + > +/** > + * Default implementation of {@link OpenOfficeServerConfiguration}. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public class DefaultOpenOfficeServerConfiguration implements OpenOfficeServerConfiguration, Initializable > +{ > + /** > + * Path to openoffice server installation. > + */ > + private String homePath = OfficeUtils.getDefaultOfficeHome().getAbsolutePath(); > + > + /** > + * Path to openoffice execution profile. > + */ > + private String profilePath = OfficeUtils.getDefaultProfileDir().getAbsolutePath(); > + > + /** > + * Maximum number of simultaneous conversion tasks to be handled by a single oo process instance. > + */ > + private int maxTasksPerProcess = 50; > + > + /** > + * Timeout for document conversion tasks. > + */ > + private long taskExecutionTimeout = 30000; > + > + /** > + * The {@link ConfigurationManager} component. > + */ > + private ConfigurationManager configurationManager; > + > + /** > + * The {@link ConfigurationSourceCollection} component. > + */ > + private ConfigurationSourceCollection sourceCollection; > + > + /** > + * {@inheritDoc} > + */ > + public void initialize() throws InitializationException > + { > + this.configurationManager.initializeConfiguration(this, this.sourceCollection.getConfigurationSources(), > + "openoffice"); > + } > + > + /** > + * {@inheritDoc} > + */ > + public String getHomePath() > + { > + return homePath; > + } > + > + /** > + * @param homePath path to openoffice server installation. > + */ > + public void setHomePath(String homePath) > + { > + this.homePath = homePath; > + } > + > + /** > + * {@inheritDoc} > + */ > + public String getProfilePath() > + { > + return profilePath; > + } > + > + /** > + * @param profilePath path to openoffice execution profile. > + */ > + public void setProfilePath(String profilePath) > + { > + this.profilePath = profilePath; > + } > + > + /** > + * {@inheritDoc} > + */ > + public int getMaxTasksPerProcess() > + { > + return maxTasksPerProcess; > + } > + > + /** > + * @param maxTasksPerProcess maximum number of simultaneous conversion tasks to be handled by a single oo process > + * instance. > + */ > + public void setMaxTasksPerProcess(int maxTasksPerProcess) > + { > + this.maxTasksPerProcess = maxTasksPerProcess; > + } > + > + /** > + * {@inheritDoc} > + */ > + public long getTaskExecutionTimeout() > + { > + return taskExecutionTimeout; > + } > + > + /** > + * @param taskExecutionTimeout timeout for document conversion tasks. > + */ > + public void setTaskExecutionTimeout(long taskExecutionTimeout) > + { > + this.taskExecutionTimeout = taskExecutionTimeout; > + } > +} > > Modified: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/transformer/OfficeToHtmlTransformer.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/transformer/OfficeToHtmlTransformer.java 2009-03-05 18:26:09 UTC (rev 17372) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/internal/transformer/OfficeToHtmlTransformer.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -24,24 +24,20 @@ > import java.io.FileOutputStream; > import java.io.IOException; > import java.io.UnsupportedEncodingException; > -import java.net.ConnectException; > > +import net.sf.jodconverter.DefaultDocumentFormatRegistry; > +import net.sf.jodconverter.DocumentFormat; > +import net.sf.jodconverter.DocumentFormatRegistry; > + > import org.xwiki.component.logging.AbstractLogEnabled; > import org.xwiki.component.phase.Initializable; > import org.xwiki.component.phase.InitializationException; > import org.xwiki.officeimporter.OfficeImporterContext; > import org.xwiki.officeimporter.OfficeImporterException; > import org.xwiki.officeimporter.internal.OfficeImporterFileStorage; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManager; > import org.xwiki.officeimporter.transformer.DocumentTransformer; > > -import com.artofsolving.jodconverter.DefaultDocumentFormatRegistry; > -import com.artofsolving.jodconverter.DocumentConverter; > -import com.artofsolving.jodconverter.DocumentFormat; > -import com.artofsolving.jodconverter.DocumentFormatRegistry; > -import com.artofsolving.jodconverter.openoffice.connection.OpenOfficeConnection; > -import com.artofsolving.jodconverter.openoffice.connection.SocketOpenOfficeConnection; > -import com.artofsolving.jodconverter.openoffice.converter.OpenOfficeDocumentConverter; > - > /** > * Transforms an Office Document into a corresponding Html document. > * > @@ -56,26 +52,11 @@ > public static final String DEFAULT_ENCODING = "UTF-8"; > > /** > - * The host address of the Open Office server. > + * The {@link OpenOfficeServerManager} component. > */ > - private String openOfficeServerIp; > + private OpenOfficeServerManager ooManager; > > /** > - * The port number of the the Open Office service > - */ > - private int openOfficeServerPort; > - > - /** > - * The connection to the Open Office server. > - */ > - private OpenOfficeConnection openOfficeServerConnection; > - > - /** > - * The document converter capable of transforming office documents into html. > - */ > - private DocumentConverter openOfficeDocumentConverter; > - > - /** > * Output format of this transformer. > */ > private DocumentFormat htmlFormat; > @@ -85,9 +66,8 @@ > */ > public void initialize() throws InitializationException > { > - openOfficeServerConnection = new SocketOpenOfficeConnection(openOfficeServerIp, openOfficeServerPort); > DocumentFormatRegistry formatRegistry = new DefaultDocumentFormatRegistry(); > - htmlFormat = formatRegistry.getFormatByFileExtension("html"); > + htmlFormat = formatRegistry.getFormatByExtension("html"); > } > > /** > @@ -95,19 +75,10 @@ > */ > public void transform(OfficeImporterContext importerContext) throws OfficeImporterException > { > - // Make a connection to the OO server. > - if (!openOfficeServerConnection.isConnected()) { > - try { > - openOfficeServerConnection.connect(); > - // Create an instance of the converter. > - openOfficeDocumentConverter = new OpenOfficeDocumentConverter(openOfficeServerConnection); > - } catch (ConnectException ex) { > - String message = > - "Could not connect to OpenOffice server at " + openOfficeServerIp + ":" + openOfficeServerPort; > - getLogger().error(message, ex); > - throw new OfficeImporterException(message, ex); > - } > - } > + // Make sure the openoffice server is running. > + if (ooManager.getServerState() != OpenOfficeServerManager.ServerState.RUNNING) { > + throw new OfficeImporterException("OpenOffice server is unavailable."); > + } > // Prepare the temporary directory structure. > OfficeImporterFileStorage storage = > new OfficeImporterFileStorage("xwiki-office-importer-" + importerContext.getCurrentUser()); > @@ -123,8 +94,7 @@ > throw new OfficeImporterException(message, ex); > } > // Make the conversion. > - openOfficeDocumentConverter.convert(storage.getInputFile(), importerContext.getSourceFormat(), storage > - .getOutputFile(), htmlFormat); > + ooManager.getDocumentConverter().convert(storage.getInputFile(), storage.getOutputFile(), htmlFormat); > // Collect the output into context. First the html output. > try { > FileInputStream fis = new FileInputStream(storage.getOutputFile()); > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManager.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManager.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManager.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,117 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.openoffice; > + > +import net.sf.jodconverter.OfficeDocumentConverter; > + > +/** > + * Component interface for managing the oo server process. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public interface OpenOfficeServerManager > +{ > + /** > + * This component's role, used when code needs to look it up. > + */ > + String ROLE = OpenOfficeServerManager.class.getName(); > + > + /** > + * Enum type used to represent the state of the oo server process. > + * > + * @version $Id$ > + * @since 1.9M1 > + */ > + enum ServerState > + { > + /** > + * Running. > + */ > + RUNNING("Running"), > + > + /** > + * Not running. > + */ > + STOPPED("Stopped"), > + > + /** > + * Error condition. > + */ > + ERROR("Error"); > + > + /** > + * Description of current server state. > + */ > + private String stateDescription; > + > + /** > + * Enum constructor. > + * > + * @param stateDescription description of current server state. > + */ > + private ServerState(String stateDescription) > + { > + this.stateDescription = stateDescription; > + } > + > + /** > + * @return a string with the description of current server state. > + */ Why not override toString()? > + public String getDescription() > + { > + return this.stateDescription; > + } > + } > + > + /** > + * @return path to openoffice server installation. > + */ > + String getOfficeHome(); > + > + /** > + * @return path to openoffice execution profile. > + */ > + String getOfficeProfile(); > + > + /** > + * @return current state of oo server process. > + */ > + ServerState getServerState(); > + > + /** > + * Starts the oo server with configuration details from xwiki.properties file (or default values). > + * > + * @throws OOServerManagerException if OpenOfficeServerManagerExceptioner start operation fails. > + */ > + void startServer() throws OpenOfficeServerManagerException; > + > + /** > + * Terminates the oo server instance. > + * > + * @throws OOServerManagerException if OpenOfficeServerManagerExceptioner stop operation fails. > + */ > + void stopServer() throws OpenOfficeServerManagerException; > + > + /** > + * @return a {@link OfficeDocumentConverter} associated with this oo server process. > + */ > + OfficeDocumentConverter getDocumentConverter(); > +} > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManagerException.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManagerException.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/OpenOfficeServerManagerException.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,55 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.openoffice; > + > +/** > + * Represents exceptions encountered during operations related to openoffice server management. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public class OpenOfficeServerManagerException extends Exception > +{ > + /** > + * Class version. > + */ > + private static final long serialVersionUID = 4165514722538231532L; > + > + /** > + * Constructs a new {@link OpenOfficeServerManagerException}. > + * > + * @param message the string explaining the error. > + * @param throwable the {@link Throwable} which is the cause of this exception. > + */ > + public OpenOfficeServerManagerException(String message, Throwable throwable) > + { > + super(message, throwable); > + } > + > + /** > + * Constructs a new {@link OpenOfficeServerManagerException}. > + * > + * @param throwable the {@link Throwable} which is the cause of this exception. > + */ > + public OpenOfficeServerManagerException(Throwable throwable) > + { > + super(throwable); > + } > +} > > Added: platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/configuration/OpenOfficeServerConfiguration.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/configuration/OpenOfficeServerConfiguration.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/main/java/org/xwiki/officeimporter/openoffice/configuration/OpenOfficeServerConfiguration.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,58 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.openoffice.configuration; > + > +/** > + * Configuration properties for office importer module. > + * <p> in in > + * These configuration properties are defined in in XWiki's global configuration file using the prefix of > + * "officeimporter". > + * </p> > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public interface OpenOfficeServerConfiguration > +{ > + /** > + * This component's role, used when code needs to look it up. > + */ > + String ROLE = OpenOfficeServerConfiguration.class.getName(); > + > + /** > + * @return path to openoffice server installation. > + */ > + String getHomePath(); > + > + /** > + * @return path to openoffice execution profile. > + */ > + String getProfilePath(); > + > + /** > + * @return maximum number of simultaneous conversion tasks to be handled by a single oo process instance. > + */ > + int getMaxTasksPerProcess(); > + > + /** > + * @return timeout for document conversion tasks. > + */ > + long getTaskExecutionTimeout(); > +} > > Modified: platform/core/trunk/xwiki-officeimporter/src/main/resources/META-INF/plexus/components.xml > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/main/resources/META-INF/plexus/components.xml 2009-03-05 18:26:09 UTC (rev 17372) > +++ platform/core/trunk/xwiki-officeimporter/src/main/resources/META-INF/plexus/components.xml 2009-03-06 09:44:22 UTC (rev 17373) > @@ -206,10 +206,11 @@ > <role>org.xwiki.officeimporter.transformer.DocumentTransformer</role> > <role-hint>officetohtml</role-hint> > <implementation>org.xwiki.officeimporter.internal.transformer.OfficeToHtmlTransformer</implementation> > - <configuration> > - <openOfficeServerIp>127.0.0.1</openOfficeServerIp> > - <openOfficeServerPort>8100</openOfficeServerPort> > - </configuration> > + <requirements> > + <requirement> > + <role>org.xwiki.officeimporter.openoffice.OpenOfficeServerManager</role> > + </requirement> > + </requirements> > <instantiation-strategy>singleton</instantiation-strategy> > </component> > <component> > @@ -277,6 +278,33 @@ > </requirement> > </requirements> > </component> > +  > + <component> > + <role>org.xwiki.officeimporter.openoffice.configuration.OpenOfficeServerConfiguration</role> > + <role-hint>default</role-hint> > + <implementation>org.xwiki.officeimporter.internal.openoffice.configuration.DefaultOpenOfficeServerConfiguration</implementation> > + <requirements> > + <requirement> > + <role>org.xwiki.configuration.ConfigurationManager</role> > + <role-hint>default</role-hint> > + </requirement> > + <requirement> > + <role>org.xwiki.configuration.ConfigurationSourceCollection</role> > + <role-hint>default</role-hint> > + </requirement> > + </requirements> > + </component> > +  > + <component> > + <role>org.xwiki.officeimporter.openoffice.OpenOfficeServerManager</role> > + <role-hint>default</role-hint> > + <implementation>org.xwiki.officeimporter.internal.openoffice.DefaultOpenOfficeServerManager</implementation> > + <requirements> > + <requirement> > + <role>org.xwiki.officeimporter.openoffice.configuration.OpenOfficeServerConfiguration</role> > + </requirement> > + </requirements> > + </component> >  > <component> > <role>org.xwiki.velocity.VelocityContextInitializer</role> > @@ -292,5 +320,19 @@ > </requirement> > </requirements> > </component> > + <component> > + <role>org.xwiki.velocity.VelocityContextInitializer</role> > + <role-hint>openoffice</role-hint> > + <implementation>org.xwiki.officeimporter.internal.openoffice.OpenOfficeServerManagerVelocityContextInitializer</implementation> > + <instantiation-strategy>singleton</instantiation-strategy> > + <requirements> > + <requirement> > + <role>org.xwiki.bridge.DocumentAccessBridge</role> > + </requirement> > + <requirement> > + <role>org.xwiki.officeimporter.openoffice.OpenOfficeServerManager</role> > + </requirement> > + </requirements> > + </component> > </components> > </component-set> > \ No newline at end of file > > Added: platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManagerTest.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManagerTest.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/DefaultOpenOfficeServerManagerTest.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,61 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.internal.openoffice; > + > +import net.sf.jodconverter.office.OfficeUtils; > + > +import org.xwiki.officeimporter.internal.MockDocumentAccessBridge; > +import org.xwiki.officeimporter.openoffice.OpenOfficeServerManager; > + > +import com.xpn.xwiki.test.AbstractXWikiComponentTestCase; > + > +/** > + * Test case for {@link DefaultOpenOfficeServerManager}. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public class DefaultOpenOfficeServerManagerTest extends AbstractXWikiComponentTestCase > +{ > + /** > + * The {@link OpenOfficeServerManager} component. > + */ > + private OpenOfficeServerManager manager; > + > + /** > + * {@inheritDoc} > + */ > + protected void setUp() throws Exception > + { > + getComponentManager().registerComponentDescriptor(MockDocumentAccessBridge.getComponentDescriptor()); > + super.setUp(); > + manager = (OpenOfficeServerManager) getComponentManager().lookup(OpenOfficeServerManager.ROLE, "default"); > + } > + > + /** > + * Tests the initial status of the oo server manager. > + */ > + public void testInitialStatus() > + { > + assertEquals(OfficeUtils.getDefaultOfficeHome().getAbsolutePath(), manager.getOfficeHome()); > + assertEquals(OfficeUtils.getDefaultProfileDir().getAbsolutePath(), manager.getOfficeProfile()); > + assertEquals(OpenOfficeServerManager.ServerState.STOPPED, manager.getServerState()); > + } > +} > > Added: platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeConfigurationTest.java > =================================================================== > --- platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeConfigurationTest.java (rev 0) > +++ platform/core/trunk/xwiki-officeimporter/src/test/java/org/xwiki/officeimporter/internal/openoffice/configuration/DefaultOpenOfficeConfigurationTest.java 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,63 @@ > +/* > + * See the NOTICE file distributed with this work for additional > + * information regarding copyright ownership. > + * > + * This is free software; you can redistribute it and/or modify it > + * under the terms of the GNU Lesser General Public License as > + * published by the Free Software Foundation; either version 2.1 of > + * the License, or (at your option) any later version. > + * > + * This software is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * Lesser General Public License for more details. > + * > + * You should have received a copy of the GNU Lesser General Public > + * License along with this software; if not, write to the Free > + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA > + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. > + */ > +package org.xwiki.officeimporter.internal.openoffice.configuration; > + > +import net.sf.jodconverter.office.OfficeUtils; > + > +import org.xwiki.officeimporter.internal.MockDocumentAccessBridge; > +import org.xwiki.officeimporter.internal.openoffice.configuration.DefaultOpenOfficeServerConfiguration; > +import org.xwiki.officeimporter.openoffice.configuration.OpenOfficeServerConfiguration; > + > +import com.xpn.xwiki.test.AbstractXWikiComponentTestCase; > + > +/** > + * Test case for {@link DefaultOpenOfficeServerConfiguration}. > + * > + * @version $Id$ > + * @since 1.8RC3 > + */ > +public class DefaultOpenOfficeConfigurationTest extends AbstractXWikiComponentTestCase > +{ > + /** > + * Office importer configuration. > + */ > + private OpenOfficeServerConfiguration configuration; > + > + /** > + * {@inheritDoc} > + */ > + protected void setUp() throws Exception > + { > + getComponentManager().registerComponentDescriptor(MockDocumentAccessBridge.getComponentDescriptor()); > + super.setUp(); > + configuration = (OpenOfficeServerConfiguration) getComponentManager().lookup(OpenOfficeServerConfiguration.ROLE, "default"); > + } > + > + /** > + * Test if default configuration values are present. > + */ > + public void testDefaultConfiguration() > + { I'm not sure that checking for specific values is a good decision. Perhaps checking for non-null, non-empty and non-negative? > + assertEquals(OfficeUtils.getDefaultOfficeHome().getAbsolutePath(), configuration.getHomePath()); > + assertEquals(OfficeUtils.getDefaultProfileDir().getAbsolutePath(), configuration.getProfilePath()); > + assertEquals(50, configuration.getMaxTasksPerProcess()); > + assertEquals(30000, configuration.getTaskExecutionTimeout()); > + } > +} > > Added: platform/xwiki-applications/trunk/officeimporter/src/main/resources/XWiki/OfficeImporterAdmin.xml > =================================================================== > --- platform/xwiki-applications/trunk/officeimporter/src/main/resources/XWiki/OfficeImporterAdmin.xml (rev 0) > +++ platform/xwiki-applications/trunk/officeimporter/src/main/resources/XWiki/OfficeImporterAdmin.xml 2009-03-06 09:44:22 UTC (rev 17373) > @@ -0,0 +1,121 @@ > +<?xml version="1.0" encoding="ISO-8859-1"?> > + > +<xwikidoc> > +<web>XWiki</web> > +<name>OfficeImporterAdmin</name> > +<language></language> > +<defaultLanguage>en</defaultLanguage> > +<translation>0</translation> > +<parent>XWiki.OfficeImporter</parent> > +<creator>XWiki.Admin</creator> > +<author>XWiki.Admin</author> > +<customClass></customClass> > +<contentAuthor>XWiki.Admin</contentAuthor> > +<creationDate>1235987110000</creationDate> > +<date>1236277246000</date> > +<contentUpdateDate>1236277246000</contentUpdateDate> > +<version>1.0</version> > +<title>OpenOffice Server</title> > +<template></template> > +<defaultTemplate></defaultTemplate> > +<validationScript></validationScript> > +<syntaxId>xwiki/1.0</syntaxId> > +<hidden>false</hidden> Is this attachment used anywhere? > +<attachment> > +<filename>icon.png</filename> > +<filesize>8674</filesize> > +<author>XWiki.Admin</author> > +<date>1236277246000</date> > +<version>1.0</version> > +<comment></comment> > +</attachment> I usually delete the tag objects from the XML file before committing. > +<object> > +<class> > +<name>XWiki.TagClass</name> > +<customClass></customClass> > +<customMapping></customMapping> > +<defaultViewSheet></defaultViewSheet> > +<defaultEditSheet></defaultEditSheet> > +<defaultWeb></defaultWeb> > +<nameField></nameField> > +<validationScript></validationScript> > +<tags> > +<cache>0</cache> > +<displayType>input</displayType> > +<multiSelect>1</multiSelect> > +<name>tags</name> > +<number>1</number> > +<prettyName>Tags</prettyName> > +<relationalStorage>1</relationalStorage> > +<separator> </separator> > +<separators> ,|</separators> > +<size>30</size> > +<unmodifiable>0</unmodifiable> > +<values></values> > +<classType>com.xpn.xwiki.objects.classes.StaticListClass</classType> > +</tags> > +</class> > +<name>XWiki.OfficeImporterAdmin</name> > +<number>0</number> > +<className>XWiki.TagClass</className> > +<guid>a0f37542-d302-44ae-afac-7808eef135ec</guid> > +<property> > +<tags/> > +</property> > +</object> > +<content> > +#set($msgServerPath=$msg.get("xe.officeimporter.openoffice.serverpath")) > +#set($msgServerProfile=$msg.get("xe.officeimporter.openoffice.serverprofile")) > +#set($msgConfiguration=$msg.get("xe.officeimporter.openoffice.configuration", [$msgServerPath, $msgServerProfile])) > +#set($msgServerState=$msg.get("xe.officeimporter.openoffice.serverstate")) > +#set($msgActions=$msg.get("xe.officeimporter.openoffice.actions")) > +#set($msgStart=$msg.get("xe.officeimporter.openoffice.actions.start")) > +#set($msgStop=$msg.get("xe.officeimporter.openoffice.actions.stop")) > +#set($msgRestart=$msg.get("xe.officeimporter.openoffice.actions.restart")) > +#set($msgUpdate=$msg.get("xe.officeimporter.openoffice.update")) > + > +#if($hasAdmin) > + #info($msgConfiguration) > + #if($request.action && $request.action=="stop") This is best done like this: #if("$!request.action" == 'stop') > + #if(!$oomanager.stopServer()) > + #error($oomanager.lastErrorMessage) > + #end > + #elseif($request.action && $request.action=="start") > + #if(!$oomanager.startServer()) > + #error($oomanager.lastErrorMessage) > + #end > + #elseif($request.action && $request.action=="restart") Since you're using it several times, you could store the action in a variable at the top: #set($currentAction = "$!request.action") > + #if(!$oomanager.stopServer()) > + #error($oomanager.lastErrorMessage) > + #elseif(!$oomanager.startServer()) > + #error($oomanager.lastErrorMessage) > + #end > + #end > +#end > + > +## Server path > +#set($serverPath=$oomanager.officeHome) > + > +## Server profile > +#set($serverProfile=$oomanager.officeProfile) > + > +## Current state > +#set($currentState=$oomanager.serverState) > + > +#if($hasAdmin) > + <form action="$xwiki.requestURL" enctype="multipart/form-data" method="post"> > +#end > +{table} What's with the paragraphs? IMO they aren't needed, just write: $msgServerPath | $serverPath > +<p>$msgServerPath</p>|<p>$serverPath</p> > +<p>$msgServerProfile</p>|<p>$serverProfile</p> > +<p>$msgServerState</p>|<p>$currentState</p> > +#if($hasAdmin) > + <p>$msgActions</p>|<p><input name="action" type="radio" value="start" checked="true"/> $msgStart<br/><input name="action" type="radio" value="stop"/> $msgStop<br/><input name="action" type="radio" value="restart"/> $msgRestart<br/></p> > +#end > +{table} > +#if($hasAdmin) > + <p><input type="submit" value="$msgUpdate"/></p> > + </form> > +#end</content> > +</xwikidoc> > \ No newline at end of file > > Modified: platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm > =================================================================== > --- platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm 2009-03-05 18:26:09 UTC (rev 17372) > +++ platform/xwiki-tools/trunk/xwiki-configuration-resources/src/main/resources/xwiki.properties.vm 2009-03-06 09:44:22 UTC (rev 17373) > @@ -42,4 +42,24 @@ > > #-# [Since 1.7M1] > #-# The cache component to use as local cache component. > -# cache.defaultLocalCache=jbosscache/local > \ No newline at end of file > +# cache.defaultLocalCache=jbosscache/local > + > +#------------------------------------------------------------------------------------- Better: Setting for the OpenOffice instance controlled by the office importer plugin. > +# OpenOffice > +#------------------------------------------------------------------------------------- > + Since 1.9M1? > +#-# [Since 1.8RC3] > +#-# Path to openoffice installation. > +# openoffice.homePath=/opt/openoffice.org3/ > + > +#-# [Since 1.8RC3] > +#-# Path to openoffice execution profile. Is this right? For me it's /home/user/.ooo3/ > +# openoffice.profilePath=/home/user/.openoffice.org/3 > + > +#-# [Since 1.8RC3] > +#-# Maximum number of simultaneous conversion tasks to be handled by a single openoffice process. > +# openoffice.maxTasksPerProcess=50 > + > +#-# [Since 1.8RC3] > +#-# Timeout for conversion tasks (in miliseconds). > +# openoffice.taskExecutionTimeout=30000 -- Sergiu Dumitriu http://purl.org/net/sergiu/

1 0

[xwiki-devs] Translation status and improvements
by Ludovic Dubost 07 Mar '09

07 Mar '09

I've improved slightly the translation app at http://l10n.xwiki.org It now supports sorting per the translation keys both for the output (the file will be nicer looking) and when translating, which will make it easier to translate keys that correspond to the same module together. In terms of status the great news is that we have the following language completed: - English (obviously) - French - German - Romanian Translators might want to have a look at the "outdated" translations. These are the already translated keys that have in the mean time changed in the english language. Outdated translations are available from the OutDated link of your language. Review is also important. It gives a second chance to check translations. Also an already translated key that has been changed and not reviewed will not be taken into account until it is reviewed. Review is accessible from the Review link of your language. In any case I'm really happy with what we have achieve. We now have 4 complete languages and we can keep track of the changes. We'll make sure this hard work is going to make it in 1.8 final release. Thanks to all for the contribution and we welcome other volunteers to tackle the other languages. Ludovic -- Ludovic Dubost Blog: http://blog.ludovic.org/ XWiki: http://www.xwiki.com Skype: ldubost GTalk: ldubost

1 0

[xwiki-devs] [VOTE] Refactoring of the image/link xwiki syntax 2.0 to be more generic
by Vincent Massol 06 Mar '09

06 Mar '09

Hi devs, Someone on the list asked how create a link and an image pointing to a relative reference (that he got using one of XWiki's API since they all return relative URL). We currently don't have a good solution for this and I'd like to propose a solution. For links ======= Make the usage of a prefix that specifies what comes next a generic feature. We have the following needs: * link to a URL * link to a relative path * link to a document * link to a mailto * link to an attachment I propose the following syntax: [[label>>prefix:value]] where prefix = one of http:, path:, doc:, mailto:, attach: In addition I propose that when there's no prefix or if the prefix is an unknown prefix then to consider that the reference is a reference to a document (equivalent to doc:). Pros: * Generic * Ability to add more prefixes later on (for ex ftp:) * Ability to have virtual wikis which have "reserved" names such as "doc", "http", "ftp", "mailto", etc since you can write [[doc:doc:Space.Page]] * Ability to write shorthand for documents in the text, such as: "This is a link doc:Space.Page" * Ability to reference relative paths and write stuff such as {{velocity}}[[label>>path:$xwiki.getURL(something)]]{{/velocity}} For images ========= * Similar to links add the ability to add prefixes in a generic way. We have the following needs: * image where the source is at a URL * image where the source is at a relative path * image where the source is in a document I propose the following syntax: image:prefix:value where prefix = one of http:, path:, attach: Note that we already support image:http://... so the proposal here is simply to make it generic. In addition I propose that when there's no prefix or if the prefix is unknown then to consider that the image is to be found in a document (equivalent to attach:). Same pros as above. Here's my +1 WDYT? Thanks -Vincent

5 8

[xwiki-devs] About GSoc 2009 project Ideas for XWiki...
by usw wickramasinghe 06 Mar '09

06 Mar '09

Hi, I'm an Computer Science Undergraduate who is currently undergoing training and engaged n the field of Open Source SOA products n development. I've recently done project work in the area of WS Security (WS [Web Services] Policy derivation from SOAP messages n some OSGI component implementatons around that..) and have fair knowlege on WS security implementations. Also also have an good understanding in Apache Axis2 Engine and been working on Apache Axis2 Engine developing Ruby Services deployment using JRuby frame work.. I would pretty much like to participate in a XWiki GSoc project for 2009 as i see this as a great opportunity to expand my understanding in this field and as a great learning experience. I had a look at previous 2008 xWiki Gsoc Ideas,Devs list issues and some jiras and i gained interest in doing some work in xWiki Core. I was particularly interested in ideas such as SAML Authentication (Reported @ xWiki JIRA by Sergiu Dumitriu<http://jira.xwiki.org/jira/browse/XWIKI-402>) , Support for AJAX Calls (Reported @ xWiki JIRA by Sergiu Dumitriu <http://jira.xwiki.org/jira/browse/XWIKI-641> ) , Xwiki Web Services integration , and would like to know whether aforementioned work (/proposals) are still available. However i would also be open to any other ideas that will be put forward by XWiki org and would like to hear about that too. BTW i still don't know whether this is too early or not to ask for project ideas n stuff ,but am certainly hoping to work with you guys if all goes well .I'm still new to XWiki engine and it's aspects but i am willing to, learn and have a good run on xWiki. Your feedback regarding this is very much appreciated..Thnx in advance . Regards, Udayanga Shaminda Wickramasinghe

3 5