- xwiki-devs@xwiki.org - lists.xwiki.org

[xwiki-devs] XWiki Enterprise and XWiki Enterprise Manager 2.7 builds ready for testing

by Sergiu Dumitriu

Hi community, The 2.7 maven builds are ready for testing at: http://maven.xwiki.org/releases/org/xwiki/enterprise/xwiki-enterprise-jetty… http://maven.xwiki.org/releases/org/xwiki/enterprise/xwiki-enterprise-insta… http://maven.xwiki.org/releases/org/xwiki/enterprise/xwiki-enterprise-web/2… http://maven.xwiki.org/releases/org/xwiki/enterprise/xwiki-enterprise-wiki/… http://maven.xwiki.org/releases/org/xwiki/manager/xwiki-enterprise-manager-… http://maven.xwiki.org/releases/org/xwiki/manager/xwiki-enterprise-manager-… http://maven.xwiki.org/releases/org/xwiki/manager/xwiki-enterprise-manager-… Please give them a try and confirm that the build is successful. Thanks, The XWiki Dev Team.

14 years, 9 months

1
0
0 0

[xwiki-devs] How to unsub from the mailing lists?

by mearvk01＠aol.com

Thanks

14 years, 9 months

1
0
0 0

[xwiki-devs] Access to XWiki authentication from gwt rpc

by Vito Impagliazzo

I have written a gwt servlet which I could successfully use in a gwt client inserted through a component into a xwiki page. I now wonder how I can check user authentication from the gwt servlet, since even though I can lookup other components, the context variables are empty. thanks Vito Impagliazzo

14 years, 9 months

2
3
0 0

[xwiki-devs] [Proposal] [UX] User Statuses

by Ecaterina Moraru (Valica)

Hi, Please provide feedback for this User Status proposal: http://incubator.myxwiki.org/xwiki/bin/view/Improvements/UserStatusProposal Other references: * Proposal Code: http://incubator.myxwiki.org/xwiki/bin/view/Improvements/UserStatusPrototype * Mail [Investigation] User Statuses: http://markmail.org/thread/65j5pc2ymmntujt3 Thanks, Caty

14 years, 9 months

9
28
0 0

Re: [xwiki-devs] [xwiki-notifications] r33583 - in platform/core/trunk: . xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge xwiki-model/src/main/java/org/xwiki/model/internal/reference xwiki-model/src/main/resources/META-INF xwiki-model/src/te

by Thomas Mortagne

On Fri, Dec 17, 2010 at 17:38, Vincent Massol <vincent(a)massol.net> wrote: > > On Dec 17, 2010, at 2:37 PM, Thomas Mortagne wrote: > >> On Fri, Dec 17, 2010 at 14:07, vmassol <platform-notifications(a)xwiki.org> wrote: >>> Author: vmassol >>> Date: 2010-12-17 14:07:48 +0100 (Fri, 17 Dec 2010) >>> New Revision: 33583 >>> >>> Added: >>> platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolver.java >>> platform/core/trunk/xwiki-model/src/test/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolverTest.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/ >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/TransformationContextTest.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParser.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/MacroContentParser.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/test/java/org/xwiki/rendering/internal/transformation/macro/MacroTransformationContextTest.java >>> Removed: >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultMacroContentParser.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/MacroContentParser.java >>> Modified: >>> platform/core/trunk/pom.xml >>> platform/core/trunk/xwiki-bridge/pom.xml >>> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/DocumentModelBridge.java >>> platform/core/trunk/xwiki-model/src/main/resources/META-INF/components.txt >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/transformation/TransformationContext.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultBoxMacro.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/macro/box/AbstractBoxMacro.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/resources/META-INF/components.txt >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacro.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/IncludeMacroTest.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/transformation/MacroTransformationContext.java >>> platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/resources/META-INF/components.txt >>> Log: >>> XWIKI-5807: Add support for relative links/images in included documents >>> >>> * First implementation. Current limitation: Doesn't work when links are generated by macros in the included document. >>> >>> Modified: platform/core/trunk/pom.xml >>> =================================================================== >>> --- platform/core/trunk/pom.xml 2010-12-17 12:25:56 UTC (rev 33582) >>> +++ platform/core/trunk/pom.xml 2010-12-17 13:07:48 UTC (rev 33583) >>> @@ -192,6 +192,8 @@ >>> <exclude>**/internal/**</exclude> >>> <exclude>**/test/**</exclude> >>>  >>> + <exclude>org/xwiki/bridge/DocumentModelBridge</exclude> >>> + <exclude>org/xwiki/rendering/macro/box/AbstractBoxMacro</exclude> >>> </excludes> >>> </configuration> >>> </plugin> >>> >>> Modified: platform/core/trunk/xwiki-bridge/pom.xml >>> =================================================================== >>> --- platform/core/trunk/xwiki-bridge/pom.xml 2010-12-17 12:25:56 UTC (rev 33582) >>> +++ platform/core/trunk/xwiki-bridge/pom.xml 2010-12-17 13:07:48 UTC (rev 33583) >>> @@ -46,6 +46,11 @@ >>> </dependency> >>> <dependency> >>> <groupId>org.xwiki.platform</groupId> >>> + <artifactId>xwiki-core-rendering-api</artifactId> >>> + <version>${project.version}</version> >>> + </dependency> >>> + <dependency> >>> + <groupId>org.xwiki.platform</groupId> >>> <artifactId>xwiki-core-observation-local</artifactId> >> >> Are you sure you need the implementation ? It should probably be >> xwiki-observation-api instead. > > Fixed, thanks (note that it was there before my commit). Indeed I misread, sorry. > > -Vincent > > [snip] > _______________________________________________ > devs mailing list > devs(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/devs > -- Thomas Mortagne

14 years, 9 months

1
0
0 0

Re: [xwiki-devs] [xwiki-notifications] r33583 - in platform/core/trunk: . xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge xwiki-model/src/main/java/org/xwiki/model/internal/reference xwiki-model/src/main/resources/META-INF xwiki-model/src/test/java/org/x

by Thomas Mortagne

On Fri, Dec 17, 2010 at 14:07, vmassol <platform-notifications(a)xwiki.org> wrote: > Author: vmassol > Date: 2010-12-17 14:07:48 +0100 (Fri, 17 Dec 2010) > New Revision: 33583 > > Added: > platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolver.java > platform/core/trunk/xwiki-model/src/test/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolverTest.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/ > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/TransformationContextTest.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParser.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/MacroContentParser.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/test/java/org/xwiki/rendering/internal/transformation/macro/MacroTransformationContextTest.java > Removed: > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultMacroContentParser.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/MacroContentParser.java > Modified: > platform/core/trunk/pom.xml > platform/core/trunk/xwiki-bridge/pom.xml > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/DocumentModelBridge.java > platform/core/trunk/xwiki-model/src/main/resources/META-INF/components.txt > platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/transformation/TransformationContext.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultBoxMacro.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/macro/box/AbstractBoxMacro.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/resources/META-INF/components.txt > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacro.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/IncludeMacroTest.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/transformation/MacroTransformationContext.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/resources/META-INF/components.txt > Log: > XWIKI-5807: Add support for relative links/images in included documents > > * First implementation. Current limitation: Doesn't work when links are generated by macros in the included document. > > Modified: platform/core/trunk/pom.xml > =================================================================== > --- platform/core/trunk/pom.xml 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/pom.xml 2010-12-17 13:07:48 UTC (rev 33583) > @@ -192,6 +192,8 @@ > <exclude>**/internal/**</exclude> > <exclude>**/test/**</exclude> >  > + <exclude>org/xwiki/bridge/DocumentModelBridge</exclude> > + <exclude>org/xwiki/rendering/macro/box/AbstractBoxMacro</exclude> > </excludes> > </configuration> > </plugin> > > Modified: platform/core/trunk/xwiki-bridge/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-bridge/pom.xml 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-bridge/pom.xml 2010-12-17 13:07:48 UTC (rev 33583) > @@ -46,6 +46,11 @@ > </dependency> > <dependency> > <groupId>org.xwiki.platform</groupId> > + <artifactId>xwiki-core-rendering-api</artifactId> > + <version>${project.version}</version> > + </dependency> > + <dependency> > + <groupId>org.xwiki.platform</groupId> > <artifactId>xwiki-core-observation-local</artifactId> Are you sure you need the implementation ? It should probably be xwiki-observation-api instead. > <version>${pom.version}</version> > </dependency> > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/DocumentModelBridge.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/DocumentModelBridge.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/DocumentModelBridge.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -20,6 +20,7 @@ > package org.xwiki.bridge; > > import org.xwiki.model.reference.DocumentReference; > +import org.xwiki.rendering.syntax.Syntax; > > /** > * Exposes methods for accessing Documents. This is temporary until we remodel the Model classes and the Document > @@ -78,19 +79,28 @@ > /** > * @return the Syntax id representing the syntax used for the current document. For example "xwiki/1.0" represents > * the first version XWiki syntax while "xwiki/2.0" represents version 2.0 of the XWiki Syntax. > + * @deprecated since 3.0M1 use {@link #getSyntax()} instead > */ > + @Deprecated > String getSyntaxId(); > - > + > /** > + * @return the Syntax id representing the syntax used for the current document. For example "xwiki/1.0" represents > + * the first version XWiki syntax while "xwiki/2.0" represents version 2.0 of the XWiki Syntax. > + * @since 3.0M1 > + */ > + Syntax getSyntax(); > + > + /** > * @return the page to which the document belongs to (eg "WebHome") > - * @deprecated since 2.2M1 use {@link #getDocumentReference()} instead > + * @deprecated since 2.2M1 use {@link #getDocumentReference()} instead > */ > @Deprecated > String getPageName(); > > /** > * @return the space to which the document belongs to (eg "Main") > - * @deprecated since 2.2M1 use {@link #getDocumentReference()} instead > + * @deprecated since 2.2M1 use {@link #getDocumentReference()} instead > */ > @Deprecated > String getSpaceName(); > > Added: platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolver.java > =================================================================== > --- platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolver.java (rev 0) > +++ platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolver.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -0,0 +1,52 @@ > +/* [...] > + */ > +package org.xwiki.model.internal.reference; > + > +import org.xwiki.component.annotation.Component; > +import org.xwiki.component.annotation.Requirement; > +import org.xwiki.model.EntityType; > +import org.xwiki.model.reference.AttachmentReference; > +import org.xwiki.model.reference.AttachmentReferenceResolver; > +import org.xwiki.model.reference.EntityReferenceResolver; > + > +/** > + * Specialized version of {@link org.xwiki.model.reference.EntityReferenceResolver} which can be considered a helper > + * component to resolve {@link org.xwiki.model.reference.AttachmentReference} objects from their string representation. > + * The behavior is the one defined in {@link org.xwiki.model.internal.reference.ExplicitStringEntityReferenceResolver}. > + * > + * @version $Id$ > + * @since 3.0M1 > + */ > +@Component("explicit") > +public class ExplicitStringAttachmentReferenceResolver implements AttachmentReferenceResolver<String> > +{ > + @Requirement("explicit") > + private EntityReferenceResolver<String> entityReferenceResolver; > + > + /** > + * {@inheritDoc} > + * @see org.xwiki.model.reference.AttachmentReferenceResolver#resolve > + */ > + public AttachmentReference resolve(String attachmentReferenceRepresentation, Object... parameters) > + { > + return new AttachmentReference(this.entityReferenceResolver.resolve( > + attachmentReferenceRepresentation, EntityType.ATTACHMENT, parameters)); > + } > +} > [..] > > Added: platform/core/trunk/xwiki-model/src/test/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolverTest.java > =================================================================== > --- platform/core/trunk/xwiki-model/src/test/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolverTest.java (rev 0) > +++ platform/core/trunk/xwiki-model/src/test/java/org/xwiki/model/internal/reference/ExplicitStringAttachmentReferenceResolverTest.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -0,0 +1,58 @@ > +/* [...] > + */ > +package org.xwiki.model.internal.reference; > + > +import org.junit.Assert; > +import org.junit.Before; > +import org.junit.Test; > +import org.xwiki.component.util.ReflectionUtils; > +import org.xwiki.model.reference.AttachmentReference; > +import org.xwiki.model.reference.AttachmentReferenceResolver; > +import org.xwiki.model.reference.DocumentReference; > +import org.xwiki.model.reference.DocumentReferenceResolver; > + > +/** > + * Unit tests for {@link org.xwiki.model.internal.reference.ExplicitStringAttachmentReferenceResolver}. > + * > + * @version $Id$ > + * @since 3.0M1 > + */ > +public class ExplicitStringAttachmentReferenceResolverTest > +{ > + private AttachmentReferenceResolver<String> resolver; > + > + @Before > + public void setUp() throws Exception > + { > + this.resolver = new ExplicitStringAttachmentReferenceResolver(); > + ReflectionUtils.setFieldValue(this.resolver, "entityReferenceResolver", > + new ExplicitStringEntityReferenceResolver()); > + } > + > + @Test > + public void testResolveWithExplicitAttachmentReference() > + { > + DocumentReference documentReference = new DocumentReference("wiki", "space", "page"); > + AttachmentReference reference = this.resolver.resolve("", new AttachmentReference("file", documentReference)); > + > + Assert.assertEquals("file", reference.getName()); > + Assert.assertEquals(documentReference, reference.getDocumentReference()); > + } > +} > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/transformation/TransformationContext.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/transformation/TransformationContext.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/main/java/org/xwiki/rendering/transformation/TransformationContext.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -28,7 +28,7 @@ > * @version $Id$ > * @since 2.4M1 > */ > -public class TransformationContext > +public class TransformationContext implements Cloneable > { > /** > * The complete {@link XDOM} of the content currently being transformed. > @@ -117,4 +117,22 @@ > { > return syntax; > } > + > + /** > + * {@inheritDoc} > + * @see Object#clone() > + */ > + @Override > + public TransformationContext clone() > + { > + TransformationContext newContext; > + try { > + newContext = (TransformationContext) super.clone(); > + } catch (CloneNotSupportedException e) { > + // Should never happen > + throw new RuntimeException("Failed to clone object", e); > + } > + > + return newContext; > + } > } > > Added: platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/TransformationContextTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/TransformationContextTest.java (rev 0) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-api/src/test/java/org/xwiki/rendering/transformation/TransformationContextTest.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -0,0 +1,54 @@ > +/* [...] > + */ > +package org.xwiki.rendering.transformation; > + > +import java.util.Arrays; > + > +import org.junit.Test; > +import org.junit.Assert; > +import org.xwiki.rendering.block.Block; > +import org.xwiki.rendering.block.WordBlock; > +import org.xwiki.rendering.block.XDOM; > +import org.xwiki.rendering.syntax.Syntax; > + > +/** > + * Unit tests for {@link TransformationContext}. > + * > + * @version $Id$ > + * @since 3.0M1 > + */ > +public class TransformationContextTest > +{ > + @Test > + public void testClone() > + { > + TransformationContext context = new TransformationContext(); > + context.setId("id"); > + context.setSyntax(Syntax.XWIKI_2_0); > + XDOM xdom = new XDOM(Arrays.<Block>asList(new WordBlock("test"))); > + context.setXDOM(xdom); > + > + TransformationContext newContext = context.clone(); > + Assert.assertNotSame(context, newContext); > + Assert.assertEquals("id", newContext.getId()); > + Assert.assertEquals(Syntax.XWIKI_2_0, newContext.getSyntax()); > + Assert.assertEquals(xdom, newContext.getXDOM()); > + } > +} > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultBoxMacro.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultBoxMacro.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultBoxMacro.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -67,6 +67,7 @@ > protected List<Block> parseContent(P parameters, String content, MacroTransformationContext context) > throws MacroExecutionException > { > - return getMacroContentParser().parse(content, context.getSyntax(), context.isInline()); > + // Don't execute transformations explicitly. They'll be executed on the generated content later on. > + return getMacroContentParser().parse(content, context, false, context.isInline()); > } > } > > Deleted: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultMacroContentParser.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultMacroContentParser.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultMacroContentParser.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -1,88 +0,0 @@ > -/* [...] > - */ > -package org.xwiki.rendering.internal.macro.box; > - > -import java.io.StringReader; > -import java.util.List; > - > -import org.xwiki.component.annotation.Component; > -import org.xwiki.component.annotation.Requirement; > -import org.xwiki.component.manager.ComponentLookupException; > -import org.xwiki.component.manager.ComponentManager; > -import org.xwiki.rendering.block.Block; > -import org.xwiki.rendering.macro.MacroExecutionException; > -import org.xwiki.rendering.parser.Parser; > -import org.xwiki.rendering.syntax.Syntax; > -import org.xwiki.rendering.util.ParserUtils; > - > -/** > - * Default implementation for {@link org.xwiki.rendering.internal.macro.box.MacroContentParser}. > - * > - * @version $Id$ > - * @since 2.6RC1 > - */ > -@Component > -public class DefaultMacroContentParser implements MacroContentParser > -{ > - /** > - * Used to look up the syntax parser to use for parsing the content. > - */ > - @Requirement > - private ComponentManager componentManager; > - > - /** > - * Utility to remove the top level paragraph. > - */ > - private ParserUtils parserUtils = new ParserUtils(); > - > - /** > - * {@inheritDoc} > - * @see MacroContentParser#parse(String, org.xwiki.rendering.syntax.Syntax, boolean) > - */ > - public List<Block> parse(String content, Syntax syntax, boolean removeTopLevelParagraph) > - throws MacroExecutionException > - { > - try { > - List<Block> blocks = getSyntaxParser(syntax).parse(new StringReader(content)).getChildren(); > - if (removeTopLevelParagraph) { > - this.parserUtils.removeTopLevelParagraph(blocks); > - } > - return blocks; > - } catch (Exception e) { > - throw new MacroExecutionException("Failed to parse content [" + content + "]", e); > - } > - } > - > - /** > - * Get the parser for the current syntax. > - * > - * @param syntax the current syntax of the title content > - * @return the parser for the current syntax > - * @throws org.xwiki.rendering.macro.MacroExecutionException Failed to find source parser. > - */ > - private Parser getSyntaxParser(Syntax syntax) throws MacroExecutionException > - { > - try { > - return this.componentManager.lookup(Parser.class, syntax.toIdString()); > - } catch (ComponentLookupException e) { > - throw new MacroExecutionException("Failed to find source parser for syntax [" + syntax + "]", e); > - } > - } > -} > > Deleted: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/MacroContentParser.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/MacroContentParser.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/MacroContentParser.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -1,49 +0,0 @@ > -/* [...] > - */ > -package org.xwiki.rendering.internal.macro.box; > - > -import java.util.List; > - > -import org.xwiki.component.annotation.ComponentRole; > -import org.xwiki.rendering.block.Block; > -import org.xwiki.rendering.macro.MacroExecutionException; > -import org.xwiki.rendering.syntax.Syntax; > - > -/** > - * Parses content of a macro field (parameter, macro content) in a given syntax. > - * > - * @version $Id$ > - * @since 2.6RC1 > - */ > -@ComponentRole > -public interface MacroContentParser > -{ > - /** > - * Parses content of a macro field (parameter, macro content) in a given syntax and optionally remove the top level > - * paragraph. > - * > - * @param content the content to parse > - * @param syntax the syntax in which the content is written in > - * @param removeTopLevelParagraph whether the top level paragraph should be removed after parsing > - * @return the result as a {@link org.xwiki.rendering.block.Block}s > - * @throws MacroExecutionException in case of a parsing error > - */ > - List<Block> parse(String content, Syntax syntax, boolean removeTopLevelParagraph) throws MacroExecutionException; > -} > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/macro/box/AbstractBoxMacro.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/macro/box/AbstractBoxMacro.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/macro/box/AbstractBoxMacro.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -31,7 +31,7 @@ > import org.xwiki.rendering.block.GroupBlock; > import org.xwiki.rendering.block.ImageBlock; > import org.xwiki.rendering.block.NewLineBlock; > -import org.xwiki.rendering.internal.macro.box.MacroContentParser; > +import org.xwiki.rendering.internal.macro.MacroContentParser; > import org.xwiki.rendering.listener.Format; > import org.xwiki.rendering.listener.reference.ResourceReference; > import org.xwiki.rendering.listener.reference.ResourceType; > @@ -137,7 +137,8 @@ > } > // we add the title, if there is one > if (!StringUtils.isEmpty(titleParameter)) { > - boxBlock.addChildren(this.contentParser.parse(titleParameter, context.getSyntax(), true)); > + // Don't execute transformations explicitly. They'll be executed on the generated content later on. > + boxBlock.addChildren(this.contentParser.parse(titleParameter, context, false, true)); > } > if (titleBlockList != null) { > boxBlock.addChildren(titleBlockList); > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/resources/META-INF/components.txt > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/resources/META-INF/components.txt 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/resources/META-INF/components.txt 2010-12-17 13:07:48 UTC (rev 33583) > @@ -1,2 +1 @@ > -org.xwiki.rendering.internal.macro.box.DefaultBoxMacro > -org.xwiki.rendering.internal.macro.box.DefaultMacroContentParser > \ No newline at end of file > +org.xwiki.rendering.internal.macro.box.DefaultBoxMacro > \ No newline at end of file > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacro.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacro.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/main/java/org/xwiki/rendering/internal/macro/include/IncludeMacro.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -19,35 +19,37 @@ > */ > package org.xwiki.rendering.internal.macro.include; > > -import java.io.StringReader; > import java.util.HashMap; > import java.util.List; > import java.util.Map; > > import org.xwiki.bridge.DocumentAccessBridge; > +import org.xwiki.bridge.DocumentModelBridge; > import org.xwiki.component.annotation.Component; > import org.xwiki.component.annotation.Requirement; > import org.xwiki.component.manager.ComponentManager; > import org.xwiki.context.Execution; > import org.xwiki.context.ExecutionContext; > import org.xwiki.context.ExecutionContextManager; > +import org.xwiki.model.reference.AttachmentReferenceResolver; > import org.xwiki.model.reference.DocumentReference; > import org.xwiki.model.reference.DocumentReferenceResolver; > +import org.xwiki.model.reference.EntityReference; > import org.xwiki.model.reference.EntityReferenceSerializer; > import org.xwiki.rendering.block.Block; > +import org.xwiki.rendering.block.ImageBlock; > import org.xwiki.rendering.block.LinkBlock; > import org.xwiki.rendering.block.MacroMarkerBlock; > import org.xwiki.rendering.block.XDOM; > +import org.xwiki.rendering.internal.macro.MacroContentParser; > import org.xwiki.rendering.listener.reference.ResourceReference; > import org.xwiki.rendering.listener.reference.ResourceType; > import org.xwiki.rendering.macro.AbstractMacro; > import org.xwiki.rendering.macro.MacroExecutionException; > import org.xwiki.rendering.macro.include.IncludeMacroParameters; > import org.xwiki.rendering.macro.include.IncludeMacroParameters.Context; > -import org.xwiki.rendering.parser.Parser; > +import org.xwiki.rendering.syntax.Syntax; > import org.xwiki.rendering.transformation.MacroTransformationContext; > -import org.xwiki.rendering.transformation.Transformation; > -import org.xwiki.rendering.transformation.TransformationContext; > > /** > * @version $Id$ > @@ -86,14 +88,24 @@ > private DocumentAccessBridge documentAccessBridge; > > /** > - * Used to transform relative document links into absolute references when including a document containing links. > - * This is required otherwise the links will be resolved at render time in the context of the including document > - * instead of in the context of the included document. > + * Used to transform the passed document reference macro parameter to a typed {@link DocumentReference} object. > */ > @Requirement("current") > private DocumentReferenceResolver<String> currentDocumentReferenceResolver; > > /** > + * Used to transform relative document links into absolute references relative to the included document. > + */ > + @Requirement("explicit") > + private DocumentReferenceResolver<String> explicitDocumentReferenceResolver; > + > + /** > + * Used to transform relative attachment links into absolute references relative to the included document. > + */ > + @Requirement("explicit") > + private AttachmentReferenceResolver<String> explicitAttachmentReferenceResolver; > + > + /** > * Used to serialize resolved document links into a string again since the Rendering API only manipulates Strings > * (done voluntarily to be independent of any wiki engine and not draw XWiki-specific dependencies). > */ > @@ -101,6 +113,12 @@ > private EntityReferenceSerializer<String> defaultEntityReferenceSerializer; > > /** > + * The parser used to parse included document content. > + */ > + @Requirement > + private MacroContentParser contentParser; > + > + /** > * Default constructor. > */ > public IncludeMacro() > @@ -141,38 +159,39 @@ > public List<Block> execute(IncludeMacroParameters parameters, String content, MacroTransformationContext context) > throws MacroExecutionException > { > - String documentName = parameters.getDocument(); > - if (documentName == null) { > + // Step 1: Perform checks > + if (parameters.getDocument() == null) { > throw new MacroExecutionException( > "You must specify a 'document' parameter pointing to the document to include."); > } > > - DocumentReference documentReference = resolve(context.getCurrentMacroBlock(), documentName); > + DocumentReference includedReference = resolve(context.getCurrentMacroBlock(), parameters.getDocument()); > > if (context.getCurrentMacroBlock() != null) { > - checkRecursiveInclusion(context.getCurrentMacroBlock(), documentReference); > + checkRecursiveInclusion(context.getCurrentMacroBlock(), includedReference); > } > > - Context actualContext = parameters.getContext(); > + if (!this.documentAccessBridge.isDocumentViewable(includedReference)) { > + throw new MacroExecutionException("Current user doesn't have view rights on document [" > + + this.defaultEntityReferenceSerializer.serialize(includedReference) + "]"); > + } > > - // Retrieve the included document's content > - String includedContent; > - String includedSyntax; > + Context parametersContext = parameters.getContext(); > + > + // Step 2: Extract included document's content and syntax. > + // TODO: use macro source information to resolve document reference based on the macro source instead > + // of the context > + DocumentModelBridge documentBridge; > try { > - if (this.documentAccessBridge.isDocumentViewable(documentReference)) { > - // TODO: use macro source informations to resolve document reference besed on the macro source instead > - // of the context > - includedContent = this.documentAccessBridge.getDocumentContent(documentName); > - includedSyntax = this.documentAccessBridge.getDocumentSyntaxId(documentName); > - } else { > - throw new MacroExecutionException("Current user doesn't have view rights on document [" + documentName > - + "]"); > - } > + documentBridge = this.documentAccessBridge.getDocument(includedReference); > } catch (Exception e) { > - throw new MacroExecutionException("Failed to get content for Document [" + documentName + "]", e); > + throw new MacroExecutionException("Failed to load Document [" > + + this.defaultEntityReferenceSerializer.serialize(includedReference) + "]", e); > } > + String includedContent = documentBridge.getContent(); > + Syntax includedSyntax = documentBridge.getSyntax(); > > - List<Block> result; > + // Step 3: Parse and transform the included document's content. > > // Check the value of the "context" parameter. > // > @@ -184,30 +203,68 @@ > // if CONTEXT_CURRENT, then simply get the included page's content, parse it and return the resulting AST > // (i.e. don't apply any transformations since we don't want any Macro to be executed at this stage since they > // should be executed by the currently running Macro Transformation. > - if (actualContext == Context.NEW) { > - result = > - executeWithNewContext(documentReference, includedContent, includedSyntax, context.getTransformation()); > + List<Block> result; > + MacroTransformationContext newContext = context.clone(); > + newContext.setSyntax(includedSyntax); > + if (parametersContext == Context.NEW) { > + // Since the execution happens in a separate context use a different transformation id to ensure it's > + // isolated (for ex this will ensure that the velocity macros defined in the included document cannot > + // interfere with the macros in the including document). > + newContext.setId(this.defaultEntityReferenceSerializer.serialize(includedReference)); > + result = executeWithNewContext(includedReference, includedContent, newContext); > } else { > - result = executeWithCurrentContext(documentReference, includedContent, includedSyntax); > + result = executeWithCurrentContext(includedReference, includedContent, newContext); > } > > - // We need to handle the case when there are relative links specified in the content of the included document. > - // These link references need to be resolved against the document being included and not the including document. > - // TODO: When http://jira.xwiki.org/jira/browse/XWIKI-4802 is implemented it should be possible remove this > - // code portion and instead perform the resolution at render time, using context information. > - XDOM xdom = new XDOM(result); > + // Step 4: Modify relative references. > + resolveRelativeReferences(result, includedReference); > + > + return result; > + } > + > + /** > + * We need to handle the case when there are relative links specified in the content of the included document. > + * These link references need to be resolved against the document being included and not the including document. > + * TODO: When http://jira.xwiki.org/jira/browse/XWIKI-4802 is implemented it should be possible remove this > + * code portion and instead perform the resolution at render time, using context information. > + * > + * @param includedReference reference to the included document > + */ > + private void resolveRelativeReferences(List<Block> blocks, DocumentReference includedReference) > + { > + XDOM xdom = new XDOM(blocks); > + > + // Resolve links > for (LinkBlock block : xdom.getChildrenByType(LinkBlock.class, true)) { > - ResourceReference reference = block.getReference(); > - if (reference.getType().equals(ResourceType.DOCUMENT)) { > - // It's a link to a document, make the reference absolute > - String resolvedReference = > - this.defaultEntityReferenceSerializer.serialize(this.currentDocumentReferenceResolver.resolve( > - reference.getReference())); > - reference.setReference(resolvedReference); > + ResourceReference resourceReference = block.getReference(); > + // Make reference absolute for links to document and attachments. > + if (resourceReference.getType().equals(ResourceType.DOCUMENT) > + || resourceReference.getType().equals(ResourceType.ATTACHMENT)) > + { > + EntityReference entityReference; > + if (resourceReference.getType().equals(ResourceType.DOCUMENT)) { > + entityReference = this.explicitDocumentReferenceResolver.resolve(resourceReference.getReference(), > + includedReference); > + } else { > + entityReference = this.explicitAttachmentReferenceResolver.resolve(resourceReference.getReference(), > + includedReference); > + } > + String resolvedReference = this.defaultEntityReferenceSerializer.serialize(entityReference); > + resourceReference.setReference(resolvedReference); > } > } > > - return result; > + // Resolve images > + for (ImageBlock block : xdom.getChildrenByType(ImageBlock.class, true)) { > + ResourceReference resourceReference = block.getReference(); > + // Make reference absolute for images in documents > + if (resourceReference.getType().equals(ResourceType.ATTACHMENT)) { > + String resolvedReference = this.defaultEntityReferenceSerializer.serialize( > + this.explicitAttachmentReferenceResolver.resolve(resourceReference.getReference(), > + includedReference)); > + resourceReference.setReference(resolvedReference); > + } > + } > } > > /** > @@ -274,13 +331,12 @@ > * > * @param includedDocumentReference the name of the document to include. > * @param includedContent the content of the document to include. > - * @param includedSyntax the syntax identifier of the provided content. > - * @param transformation the macro transformation. > + * @param macroContext the transformation context to use to parse/transform the content > * @return the result of parsing and transformation of the document to include. > * @throws MacroExecutionException error when parsing content. > */ > private List<Block> executeWithNewContext(DocumentReference includedDocumentReference, String includedContent, > - String includedSyntax, Transformation transformation) throws MacroExecutionException > + MacroTransformationContext macroContext) throws MacroExecutionException > { > List<Block> result; > > @@ -293,8 +349,7 @@ > Map<String, Object> backupObjects = new HashMap<String, Object>(); > try { > this.documentAccessBridge.pushDocumentInContext(backupObjects, includedDocumentReference); > - result = > - generateIncludedPageDOM(includedDocumentReference, includedContent, includedSyntax, transformation); > + result = generateIncludedPageDOM(includedDocumentReference, includedContent, macroContext, true); > } finally { > this.documentAccessBridge.popDocumentFromContext(backupObjects); > } > @@ -315,14 +370,14 @@ > * > * @param includedDocumentReference the name of the document to include. > * @param includedContent the content of the document to include. > - * @param includedSyntax the syntax identifier of the provided content. > + * @param macroContext the transformation context to use to parse the content > * @return the result of parsing and transformation of the document to include. > * @throws MacroExecutionException error when parsing content. > */ > private List<Block> executeWithCurrentContext(DocumentReference includedDocumentReference, String includedContent, > - String includedSyntax) throws MacroExecutionException > + MacroTransformationContext macroContext) throws MacroExecutionException > { > - return generateIncludedPageDOM(includedDocumentReference, includedContent, includedSyntax, null); > + return generateIncludedPageDOM(includedDocumentReference, includedContent, macroContext, false); > } > > /** > @@ -330,30 +385,26 @@ > * > * @param includedDocumentReference the name of the document to include. > * @param includedContent the content of the document to include. > - * @param includedSyntax the syntax identifier of the provided content. > - * @param transformation the macro transformation. > + * @param macroContext the transformation context to use to parse/transform the content > + * @param transform if true then execute transformations > * @return the result of parsing and transformation of the document to include. > * @throws MacroExecutionException error when parsing content. > */ > private List<Block> generateIncludedPageDOM(DocumentReference includedDocumentReference, String includedContent, > - String includedSyntax, Transformation transformation) throws MacroExecutionException > + MacroTransformationContext macroContext, boolean transform) throws MacroExecutionException > { > - XDOM includedDom; > + List<Block> result; > try { > - Parser parser = this.componentManager.lookup(Parser.class, includedSyntax); > - includedDom = parser.parse(new StringReader(includedContent)); > > // Only run Macro transformation when the context is a new one as otherwise we need the macros in the > // included page to be added to the list of macros on the including page so that they're all sorted > // and executed in the right order. Note that this works only because the Include macro has the highest > // execution priority and is thus executed first. > - if (transformation != null) { > - transformation.transform(includedDom, new TransformationContext(includedDom, parser.getSyntax())); > - } > + result = this.contentParser.parse(includedContent, macroContext, transform, false); > } catch (Exception e) { > throw new MacroExecutionException("Failed to parse included page [" + includedDocumentReference + "]", e); > } > > - return includedDom.getChildren(); > + return result; > } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/IncludeMacroTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/IncludeMacroTest.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-include/src/test/java/org/xwiki/rendering/internal/macro/IncludeMacroTest.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -28,6 +28,8 @@ > > import org.jmock.Expectations; > import org.junit.Test; > +import org.xwiki.bridge.DocumentModelBridge; > +import org.xwiki.model.reference.AttachmentReference; > import org.xwiki.model.reference.DocumentReference; > import org.xwiki.rendering.block.Block; > import org.xwiki.rendering.block.MacroBlock; > @@ -73,77 +75,35 @@ > } > > @Test > - public void testIncludeMacroWithNewContext() throws Exception > + public void testIncludeMacroWithNewContextShowsVelocityMacrosAreIsolated() throws Exception > { > String expected = "beginDocument\n" > - + "beginMacroMarkerStandalone [velocity] [] [$myvar]\n" > + + "beginMacroMarkerStandalone [velocity] [] [#testmacro]\n" > + "beginParagraph\n" > - + "onWord [hello]\n" > + + "onSpecialSymbol [#]\n" > + + "onWord [testmacro]\n" > + "endParagraph\n" > - + "endMacroMarkerStandalone [velocity] [] [$myvar]\n" > + + "endMacroMarkerStandalone [velocity] [] [#testmacro]\n" > + "endDocument"; > > - // Since it's not in the same context, we verify that a Velocity variable set in the including page is not > - // seen in the included page. > - VelocityManager velocityManager = getComponentManager().lookup(VelocityManager.class); > - StringWriter writer = new StringWriter(); > - velocityManager.getVelocityEngine().evaluate(velocityManager.getVelocityContext(), writer, "template", > - "#set ($myvar = 'hello')"); > + // We verify that a Velocity macro set in the including page is not seen in the included page. > + List<Block> blocks = runIncludeMacroWithPreVelocity(Context.NEW, "#macro(testmacro)#end", > + "{{velocity}}#testmacro{{/velocity}}"); > > - getMockery().checking(new Expectations() {{ > - oneOf(mockSetup.bridge).isDocumentViewable(with(any(DocumentReference.class))); will(returnValue(true)); > - oneOf(mockSetup.bridge).getDocumentContent(with(any(String.class))); > - will(returnValue("{{velocity}}$myvar{{/velocity}}")); > - oneOf(mockSetup.bridge).getDocumentSyntaxId(with(any(String.class))); > - will(returnValue(Syntax.XWIKI_2_0.toIdString())); > - oneOf(mockSetup.bridge).pushDocumentInContext(with(any(Map.class)), with(any(DocumentReference.class))); > - oneOf(mockSetup.bridge).popDocumentFromContext(with(any(Map.class))); > - oneOf(mockSetup.documentReferenceResolver).resolve("wiki:Space.Page"); > - will(returnValue(new DocumentReference("wiki", "Space", "Page"))); > - }}); > - this.includeMacro.setDocumentAccessBridge(this.mockSetup.bridge); > - > - IncludeMacroParameters parameters = new IncludeMacroParameters(); > - parameters.setDocument("wiki:Space.Page"); > - parameters.setContext(Context.NEW); > - > - // Create a Macro transformation context with the Macro transformation object defined so that the include > - // macro can transform included page which is using a new context. > - MacroTransformation macroTransformation = > - (MacroTransformation) getComponentManager().lookup(Transformation.class, "macro"); > - MacroTransformationContext context = new MacroTransformationContext(); > - context.setTransformation(macroTransformation); > - > - List<Block> blocks = this.includeMacro.execute(parameters, null, context); > - > assertBlocks(expected, blocks, this.rendererFactory); > } > > @Test > - public void testIncludeMacroWithCurrentContext() throws Exception > + public void testIncludeMacroWithCurrentContextShowsVelocityMacrosAreShared() throws Exception > { > String expected = "beginDocument\n" > - + "onMacroStandalone [someMacro] []\n" > + + "onMacroStandalone [velocity] [] [#testmacro]\n" > + "endDocument"; > > - getMockery().checking(new Expectations() {{ > - oneOf(mockSetup.bridge).isDocumentViewable(with(any(DocumentReference.class))); will(returnValue(true)); > - oneOf(mockSetup.bridge).getDocumentContent(with(any(String.class))); > - will(returnValue("{{someMacro/}}")); > - oneOf(mockSetup.bridge).getDocumentSyntaxId(with(any(String.class))); > - will(returnValue(Syntax.XWIKI_2_0.toIdString())); > - oneOf(mockSetup.documentReferenceResolver).resolve("wiki:Space.Page"); > - will(returnValue(new DocumentReference("wiki", "Space", "Page"))); > - }}); > - > - this.includeMacro.setDocumentAccessBridge(mockSetup.bridge); > + // We verify that a Velocity macro set in the including page is seen in the included page. > + List<Block> blocks = runIncludeMacroWithPreVelocity(Context.CURRENT, "#macro(testmacro)#end", > + "{{velocity}}#testmacro{{/velocity}}"); > > - IncludeMacroParameters parameters = new IncludeMacroParameters(); > - parameters.setDocument("wiki:Space.Page"); > - parameters.setContext(Context.CURRENT); > - > - List<Block> blocks = this.includeMacro.execute(parameters, null, new MacroTransformationContext()); > - > assertBlocks(expected, blocks, this.rendererFactory); > } > > @@ -165,27 +125,30 @@ > * Verify that relative links are made absolute in the XDOM returned by the Include macro. > */ > @Test > - public void testIncludeMacroWhenIncludingDocumentWithRelativeLinks() throws Exception > + public void testIncludeMacroWhenIncludingDocumentWithRelativeReferences() throws Exception > { > String expected = "beginDocument\n" > + "beginParagraph\n" > - + "beginLink [Typed = [false] Type = [doc] Reference = [wiki:space.page]] [false]\n" > - + "endLink [Typed = [false] Type = [doc] Reference = [wiki:space.page]] [false]\n" > + + "beginLink [Typed = [false] Type = [doc] Reference = [includedWiki:includedSpace.page]] [false]\n" > + + "endLink [Typed = [false] Type = [doc] Reference = [includedWiki:includedSpace.page]] [false]\n" > + + "onSpace\n" > + + "beginLink [Typed = [true] Type = [attach] " > + + "Reference = [includedWiki:includedSpace.includedPage@test.png]] [false]\n" > + + "endLink [Typed = [true] Type = [attach] " > + + "Reference = [includedWiki:includedSpace.includedPage@test.png]] [false]\n" > + + "onSpace\n" > + + "onImage [Typed = [false] Type = [attach] " > + + "Reference = [includedWiki:includedSpace.includedPage@test.png]] [true]\n" > + "endParagraph\n" > + "endDocument"; > > + setUpDocumentMock("includedWiki:includedSpace.includedPage", > + new DocumentReference("includedWiki", "includedSpace", "includedPage"), > + "[[page]] [[attach:test.png]] image:test.png"); > getMockery().checking(new Expectations() {{ > oneOf(mockSetup.bridge).isDocumentViewable(with(any(DocumentReference.class))); will(returnValue(true)); > - oneOf(mockSetup.bridge).getDocumentContent("includedWiki:includedSpace.includedPage"); > - will(returnValue("[[page]]")); > - oneOf(mockSetup.bridge).getDocumentSyntaxId(with(any(String.class))); > - will(returnValue(Syntax.XWIKI_2_0.toIdString())); > oneOf(mockSetup.bridge).pushDocumentInContext(with(any(Map.class)), with(any(DocumentReference.class))); > oneOf(mockSetup.bridge).popDocumentFromContext(with(any(Map.class))); > - oneOf(mockSetup.documentReferenceResolver).resolve("page"); > - will(returnValue(new DocumentReference("wiki", "space", "page"))); > - oneOf(mockSetup.documentReferenceResolver).resolve("includedWiki:includedSpace.includedPage"); > - will(returnValue(new DocumentReference("includedWiki", "includedSpace", "includedPage"))); > }}); > > IncludeMacroParameters parameters = new IncludeMacroParameters(); > @@ -201,32 +164,84 @@ > public void testIncludeMacroWithRecursiveInclude() throws Exception > { > getMockery().checking(new Expectations() {{ > - oneOf(mockSetup.documentReferenceResolver).resolve("wiki:Space.Page"); > - will(returnValue(new DocumentReference("wiki", "Space", "Page"))); > - oneOf(mockSetup.documentReferenceResolver).resolve("Space.Page"); > - will(returnValue(new DocumentReference("wiki", "Space", "Page"))); > + allowing(mockSetup.documentReferenceResolver).resolve("wiki:space.page"); > + will(returnValue(new DocumentReference("wiki", "space", "page"))); > + allowing(mockSetup.documentReferenceResolver).resolve("space.page"); > + will(returnValue(new DocumentReference("wiki", "space", "page"))); > }}); > - > + > this.includeMacro.setDocumentAccessBridge(mockSetup.bridge); > > - MacroBlock includeMacro = new MacroBlock("include", Collections.singletonMap("document", "wiki:Space.Page"), false); > - new MacroMarkerBlock("include", Collections.singletonMap("document", "Space.Page"), Collections.<Block>singletonList(includeMacro), false); > + MacroBlock includeMacro = > + new MacroBlock("include", Collections.singletonMap("document", "wiki:space.page"), false); > + new MacroMarkerBlock("include", Collections.singletonMap("document", "space.page"), > + Collections.<Block>singletonList(includeMacro), false); > MacroTransformationContext context = new MacroTransformationContext(); > context.setCurrentMacroBlock(includeMacro); > > IncludeMacroParameters parameters = new IncludeMacroParameters(); > - parameters.setDocument("wiki:Space.Page"); > + parameters.setDocument("wiki:space.page"); > parameters.setContext(Context.CURRENT); > > - List<Block> blocks; > try { > - blocks = this.includeMacro.execute(parameters, null, context); > - > - Assert.fail("The include macro did not checked the recusive inclusion"); > + this.includeMacro.execute(parameters, null, context); > + Assert.fail("The include macro hasn't checked the recursive inclusion"); > } catch (MacroExecutionException expected) { > if (!expected.getMessage().startsWith("Found recursive inclusion")) { > throw expected; > } > } > } > + > + private void setUpDocumentMock(final String resolve, final DocumentReference reference, final String content) > + throws Exception > + { > + final DocumentModelBridge mockDocument = getMockery().mock(DocumentModelBridge.class, resolve); > + getMockery().checking(new Expectations() {{ > + allowing(mockSetup.documentReferenceResolver).resolve(resolve); > + will(returnValue(reference)); > + allowing(mockSetup.bridge).getDocument(reference); will(returnValue(mockDocument)); > + allowing(mockDocument).getSyntax(); will(returnValue(Syntax.XWIKI_2_0)); > + allowing(mockDocument).getContent(); will(returnValue(content)); > + }}); > + } > + > + private List<Block> runIncludeMacroWithPreVelocity(Context context, String velocity, String includedContent) > + throws Exception > + { > + VelocityManager velocityManager = getComponentManager().lookup(VelocityManager.class); > + StringWriter writer = new StringWriter(); > + velocityManager.getVelocityEngine().evaluate(velocityManager.getVelocityContext(), writer, "templateId", > + velocity); > + > + return runIncludeMacro(context, includedContent); > + } > + > + private List<Block> runIncludeMacro(final Context context, String includedContent) throws Exception > + { > + setUpDocumentMock("wiki:space.page", new DocumentReference("wiki", "space", "page"), includedContent); > + getMockery().checking(new Expectations() {{ > + allowing(mockSetup.bridge).isDocumentViewable(with(any(DocumentReference.class))); will(returnValue(true)); > + // Verify that push/pop are called when context is NEW > + if (context == Context.NEW) { > + oneOf(mockSetup.bridge).pushDocumentInContext(with(any(Map.class)), with(any(DocumentReference.class))); > + oneOf(mockSetup.bridge).popDocumentFromContext(with(any(Map.class))); > + } > + }}); > + this.includeMacro.setDocumentAccessBridge(this.mockSetup.bridge); > + > + IncludeMacroParameters parameters = new IncludeMacroParameters(); > + parameters.setDocument("wiki:space.page"); > + parameters.setContext(context); > + > + // Create a Macro transformation context with the Macro transformation object defined so that the include > + // macro can transform included page which is using a new context. > + MacroTransformation macroTransformation = > + (MacroTransformation) getComponentManager().lookup(Transformation.class, "macro"); > + MacroTransformationContext macroContext = new MacroTransformationContext(); > + macroContext.setId("templateId"); > + macroContext.setTransformation(macroTransformation); > + > + return this.includeMacro.execute(parameters, null, macroContext); > + } > } > > Copied: platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParser.java (from rev 33541, platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/DefaultMacroContentParser.java) > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParser.java (rev 0) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/DefaultMacroContentParser.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -0,0 +1,105 @@ > +/* [...] > + */ > +package org.xwiki.rendering.internal.macro; > + > +import java.io.StringReader; > +import java.util.List; > + > +import org.xwiki.component.annotation.Component; > +import org.xwiki.component.annotation.Requirement; > +import org.xwiki.component.manager.ComponentLookupException; > +import org.xwiki.component.manager.ComponentManager; > +import org.xwiki.rendering.block.Block; > +import org.xwiki.rendering.block.XDOM; > +import org.xwiki.rendering.macro.MacroExecutionException; > +import org.xwiki.rendering.parser.Parser; > +import org.xwiki.rendering.syntax.Syntax; > +import org.xwiki.rendering.transformation.MacroTransformationContext; > +import org.xwiki.rendering.transformation.TransformationContext; > +import org.xwiki.rendering.util.ParserUtils; > + > +/** > + * Default implementation for {@link org.xwiki.rendering.internal.macro.MacroContentParser}. > + * > + * @version $Id$ > + * @since 3.0M1 > + */ > +@Component > +public class DefaultMacroContentParser implements MacroContentParser > +{ > + /** > + * Used to look up the syntax parser to use for parsing the content. > + */ > + @Requirement > + private ComponentManager componentManager; > + > + /** > + * Utility to remove the top level paragraph. > + */ > + private ParserUtils parserUtils = new ParserUtils(); > + > + /** > + * {@inheritDoc} > + * @see MacroContentParser#parse(String, MacroTransformationContext, boolean, boolean) > + */ > + public List<Block> parse(String content, MacroTransformationContext macroContext, boolean transform, > + boolean removeTopLevelParagraph) throws MacroExecutionException > + { > + try { > + List<Block> blocks = getSyntaxParser(macroContext.getSyntax()).parse( > + new StringReader(content)).getChildren(); > + > + if (transform && macroContext.getTransformation() != null) { > + XDOM xdom = new XDOM(blocks); > + TransformationContext txContext = new TransformationContext(xdom, macroContext.getSyntax()); > + txContext.setId(macroContext.getId()); > + try { > + macroContext.getTransformation().transform(xdom, txContext); > + } catch (Exception e) { > + throw new MacroExecutionException("Failed to perform transformation", e); > + } > + blocks = xdom.getChildren(); > + } > + > + if (removeTopLevelParagraph) { > + this.parserUtils.removeTopLevelParagraph(blocks); > + } > + return blocks; > + } catch (Exception e) { > + throw new MacroExecutionException("Failed to parse content [" + content + "]", e); > + } > + } > + > + /** > + * Get the parser for the current syntax. > + * > + * @param syntax the current syntax of the title content > + * @return the parser for the current syntax > + * @throws org.xwiki.rendering.macro.MacroExecutionException Failed to find source parser. > + */ > + private Parser getSyntaxParser(Syntax syntax) throws MacroExecutionException > + { > + try { > + return this.componentManager.lookup(Parser.class, syntax.toIdString()); > + } catch (ComponentLookupException e) { > + throw new MacroExecutionException("Failed to find source parser for syntax [" + syntax + "]", e); > + } > + } > +} > > Copied: platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/MacroContentParser.java (from rev 33541, platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/main/java/org/xwiki/rendering/internal/macro/box/MacroContentParser.java) > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/MacroContentParser.java (rev 0) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/internal/macro/MacroContentParser.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -0,0 +1,51 @@ > +/* > + * 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.rendering.internal.macro; > + > +import java.util.List; > + > +import org.xwiki.component.annotation.ComponentRole; > +import org.xwiki.rendering.block.Block; > +import org.xwiki.rendering.macro.MacroExecutionException; > +import org.xwiki.rendering.transformation.MacroTransformationContext; > + > +/** > + * Parses content of a macro field (parameter, macro content) in a given syntax. > + * > + * @version $Id$ > + * @since 3.0M1 > + */ > +@ComponentRole > +public interface MacroContentParser > +{ > + /** > + * Parses content of a macro field (parameter, macro content) in a given syntax and optionally remove the top level > + * paragraph. > + * > + * @param content the content to parse > + * @param macroContext the executing Macro context (from which to get the current syntax, etc) > + * @param transform if true then executes transformations > + * @param removeTopLevelParagraph whether the top level paragraph should be removed after parsing > + * @return the result as a {@link org.xwiki.rendering.block.Block}s > + * @throws MacroExecutionException in case of a parsing error > + */ > + List<Block> parse(String content, MacroTransformationContext macroContext, boolean transform, > + boolean removeTopLevelParagraph) throws MacroExecutionException; > +} > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/transformation/MacroTransformationContext.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/transformation/MacroTransformationContext.java 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/java/org/xwiki/rendering/transformation/MacroTransformationContext.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -29,7 +29,7 @@ > * > * @version $Id$ > */ > -public class MacroTransformationContext > +public class MacroTransformationContext implements Cloneable > { > /** > * The context of the transformation process. > @@ -181,4 +181,24 @@ > { > this.transformationContext.setId(id); > } > + > + /** > + * {@inheritDoc} > + * @see Object#clone() > + */ > + @Override > + public MacroTransformationContext clone() > + { > + MacroTransformationContext newContext; > + try { > + newContext = (MacroTransformationContext) super.clone(); > + } catch (CloneNotSupportedException e) { > + // Should never happen > + throw new RuntimeException("Failed to clone object", e); > + } > + > + newContext.transformationContext = getTransformationContext().clone(); > + > + return newContext; > + } > } > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/resources/META-INF/components.txt > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/resources/META-INF/components.txt 2010-12-17 12:25:56 UTC (rev 33582) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/main/resources/META-INF/components.txt 2010-12-17 13:07:48 UTC (rev 33583) > @@ -2,5 +2,6 @@ > org.xwiki.rendering.internal.macro.DefaultMacroCategoryManager > org.xwiki.rendering.internal.macro.DefaultMacroIdFactory > org.xwiki.rendering.internal.macro.ResourceReferenceConverter > +org.xwiki.rendering.internal.macro.DefaultMacroContentParser > org.xwiki.rendering.internal.transformation.macro.MacroTransformation > org.xwiki.rendering.internal.transformation.macro.DefaultMacroTransformationConfiguration > > Added: platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/test/java/org/xwiki/rendering/internal/transformation/macro/MacroTransformationContextTest.java > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/test/java/org/xwiki/rendering/internal/transformation/macro/MacroTransformationContextTest.java (rev 0) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-transformations/xwiki-rendering-transformation-macro/src/test/java/org/xwiki/rendering/internal/transformation/macro/MacroTransformationContextTest.java 2010-12-17 13:07:48 UTC (rev 33583) > @@ -0,0 +1,92 @@ > +/* [...] > + */ > +package org.xwiki.rendering.internal.transformation.macro; > + > +import java.util.Arrays; > +import java.util.Collections; > + > +import org.junit.Assert; > +import org.junit.Test; > +import org.xwiki.rendering.block.Block; > +import org.xwiki.rendering.block.MacroBlock; > +import org.xwiki.rendering.block.WordBlock; > +import org.xwiki.rendering.block.XDOM; > +import org.xwiki.rendering.syntax.Syntax; > +import org.xwiki.rendering.transformation.MacroTransformationContext; > +import org.xwiki.rendering.transformation.Transformation; > +import org.xwiki.rendering.transformation.TransformationContext; > +import org.xwiki.rendering.transformation.TransformationException; > + > +/** > + * Unit tests for {@link MacroTransformationContext}. > + * > + * @version $Id$ > + * @since 3.0M1 > + */ > +public class MacroTransformationContextTest > +{ > + @Test > + public void testClone() > + { > + MacroTransformationContext context = new MacroTransformationContext(); > + context.setId("id"); > + context.setInline(true); > + context.setSyntax(Syntax.XWIKI_2_0); > + > + XDOM xdom = new XDOM(Arrays.<Block>asList(new WordBlock("test1"))); > + context.setXDOM(xdom); > + > + MacroBlock macroBlock = new MacroBlock("testmacro", Collections.<String, String>emptyMap(), null, false); > + context.setCurrentMacroBlock(macroBlock); > + > + Transformation transformation = new Transformation() > + { > + public int getPriority() > + { > + throw new RuntimeException("dummy"); > + } > + > + public void transform(XDOM dom, Syntax syntax) throws TransformationException > + { > + throw new RuntimeException("dummy"); > + } > + > + public void transform(Block block, TransformationContext context) throws TransformationException > + { > + throw new RuntimeException("dummy"); > + } > + > + public int compareTo(Transformation transformation) > + { > + throw new RuntimeException("dummy"); > + } > + }; > + context.setTransformation(transformation); > + > + MacroTransformationContext newContext = context.clone(); > + Assert.assertNotSame(context, newContext); > + Assert.assertEquals("id", newContext.getId()); > + Assert.assertEquals(true, newContext.isInline()); > + Assert.assertEquals(Syntax.XWIKI_2_0, newContext.getSyntax()); > + Assert.assertEquals(xdom, newContext.getXDOM()); > + Assert.assertEquals(macroBlock, newContext.getCurrentMacroBlock()); > + Assert.assertEquals(transformation, newContext.getTransformation()); > + } > +} > > _______________________________________________ > notifications mailing list > notifications(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/notifications > -- Thomas Mortagne

14 years, 9 months

2
1
0 0

[xwiki-devs] [proposal] Deprecate XWikiAttachmentArchive#getArchive() and setArchive() and remove and make private deprecated methods in XHS

by Caleb James DeLisle

I would like to deprecate the following methods because they insert and remove content which is dependent on JRCS for parsing and generation: XWikiAttachmentArchive#getArchive() XWikiAttachmentArchive#getArchive(XWikiContext) XWikiAttachmentArchive#setArchive(byte[]) These are not easily removed because they are needed for the packaging plugin but I think we should move away from that means of import/export as it contains code (including JRCS) which is memory inefficient. Also in the 3.0 cycle, I would like to make private the following methods from XWikiHibernateStore as they are not used externally and are currently deprecated: loadXWikiProperty saveXWikiClass loadXWikiClass loadAttachmentList saveAttachmentList saveAttachment injectCustomMappingsInSessionFactory injectInSessionFactory isValidCustomMapping I would like to remove the following methods entirely as they are also deprecated and are not used at all: getBatcherStats resetBatcherStats deleteXWikiClass saveXWikiClassProperty WDYT? Caleb

14 years, 9 months

2
2
0 0

[xwiki-devs] XWiki Enterprise 2.7 Release Candidate 1 Released

by Sergiu Dumitriu

The XWiki development team is pleased to announce the release of XWiki Enterprise 2.7 Release Candidate 1. The 2.7 release is intended to be the last major release in the 2.x cycle. As such, this release has stabilization as its primary goal, and mostly provides bugfixes and minor improvements. For more information, see the full release notes at: http://www.xwiki.org/xwiki/bin/Main/ReleaseNotesXWikiEnterprise27RC1 Thanks, The XWiki dev team.

14 years, 9 months

1
0
0 0

[xwiki-devs] [Proposal] New macro: {{current}}

by Vincent Massol

Hi devs, I'd like to propose a new macro (not sure how to call it): {{current}} Usage: {{current document="wiki:space.page"}} .... nested blocks.... {{/current}} What it does: * Execute the nested blocks in the context of the passed document, i.e. it sets the passed document as the current document in the execution context. Rationale: * We have several use cases: ** When we have an include and we want to use relative references (links, images) in the included document ** On the Features page of enterprise.xwiki.org I query extensions.xwiki.org and get the data from one object field which has the format: "attach:...." (or "url:....", etc). I need to make this an image that is resolved based on the page where the information was taken from. WDYT? Thanks -Vincent

14 years, 9 months

4
9
0 0

[xwiki-devs] [Proposal] Roadmap for XE 3.0

by Vincent Massol

Hi devs, We're starting a new dev cycle (the 3.x one) and we need to start defining the roadmap for 3.0. Here are some ideas we discussed internally at XWiki SAS. Some of these already have someone assigned and some don't. If you're interested in implementing one of the ideas listed above, please reply to this mail. If you're interested in contributing some other stuff for XE 3.0 please reply to this mail too so that we can put the result of this thread on the XE roadmap page at http://enterprise.xwiki.org/xwiki/bin/view/Main/Roadmap Note that most of the issue listed are continuation of work already started. - Extension Manager - Thomas Mortagne - App within minute - Investigation - By whom? - Improved page loading - Investigation - By whom? - Use status - whom? - Right UI - whom? Sergiu? - Portlet - Marius - Multipage export - whom? - Dashboard - Anca - Admin/Profile UI improvements - Sergiu - Attachment Storage - Caleb? (+ Sergiu to help review it) Marius also expressed interest in improving support for Chrome/Safari/Opera in the WYSIWYG editor (especially for macro edition) + check how it works on IE9 + FF4. On my side, I'll try to continue working on the xwiki-action, xwiki-url and xwiki-model modules Proposed Dates: - 3.0M1: 17th of Jan 2011 - 3.0M2: 7 Feb 2011 - 3.0RC1: 21 Feb 2011 - 3.0Final: 7 March 2011 WDYT? Thanks -Vincent

14 years, 9 months

6
6
0 0

Re: [xwiki-devs] [xwiki-notifications] r33548 - platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC

by Vincent Massol

Hi Sergiu/Sorin, On Dec 15, 2010, at 8:58 PM, sdumitriu (SVN) wrote: > Author: sdumitriu > Date: 2010-12-15 20:58:05 +0100 (Wed, 15 Dec 2010) > New Revision: 33548 > > Modified: > platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClass.xml > platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassSheet.xml > platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassTemplate.xml > platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCBot.xml > platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/WebHome.xml > Log: > XAIRCBOT-2: Changed ircbot syntax logging, colouring and full 2.0 syntax > Done. > Patch from Sorin Burjan applied with some XML cleanup > > Modified: platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClass.xml > =================================================================== > --- platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClass.xml 2010-12-15 19:57:51 UTC (rev 33547) > +++ platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClass.xml 2010-12-15 19:58:05 UTC (rev 33548) > @@ -20,7 +20,7 @@ > <validationScript></validationScript> > <comment></comment> > <minorEdit>false</minorEdit> > -<syntaxId>xwiki/1.0</syntaxId> > +<syntaxId>xwiki/2.0</syntaxId> > <hidden>false</hidden> > <class> > <name>IRC.IRCArchiveClass</name> > @@ -63,7 +63,13 @@ > <classType>com.xpn.xwiki.objects.classes.TextAreaClass</classType> > </content> > </class> > -<content>#set($defaultparent = "IRC.WebHome") > +− > +<content> > +{{velocity filter="none"}} We shouldn't use this filter. It's set automatically by the converter but it's not necessary and makes the code more complex than it should be and prevents us to use indentation! There are some other cases below where it's used too. Thanks -Vincent > +#set($defaultparent = "IRC.WebHome") > #set($defaultweb = "IRC") > -#includeForm("XWiki.ClassSheet")</content> > +{{/velocity}} > + > +{{include document="XWiki.ClassSheet"/}} > +</content> > </xwikidoc> > > Modified: platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassSheet.xml > =================================================================== > --- platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassSheet.xml 2010-12-15 19:57:51 UTC (rev 33547) > +++ platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassSheet.xml 2010-12-15 19:58:05 UTC (rev 33548) > @@ -20,12 +20,10 @@ > <validationScript></validationScript> > <comment></comment> > <minorEdit>false</minorEdit> > -<syntaxId>xwiki/1.0</syntaxId> > +<syntaxId>xwiki/2.0</syntaxId> > <hidden>false</hidden> > -<content>$doc.use("IRC.IRCArchiveClass") > -{pre} > -<pre> > -$xwiki.getXMLEncoded($doc.display("content")) > -</pre> > -{/pre}</content> > -</xwikidoc> > +<content>{{velocity filter="none"}} > +{{code language="irc"}} > +$doc.getObject('IRC.IRCArchiveClass').getProperty('content').getValue() > +{{/code}} > +{{/velocity}}</content></xwikidoc> > > Modified: platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassTemplate.xml > =================================================================== > --- platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassTemplate.xml 2010-12-15 19:57:51 UTC (rev 33547) > +++ platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCArchiveClassTemplate.xml 2010-12-15 19:58:05 UTC (rev 33548) > @@ -20,7 +20,7 @@ > <validationScript></validationScript> > <comment></comment> > <minorEdit>false</minorEdit> > -<syntaxId>xwiki/1.0</syntaxId> > +<syntaxId>xwiki/2.0</syntaxId> > <hidden>false</hidden> > -<content>#includeForm("IRC.IRCArchiveClassSheet") > -</content></xwikidoc> > \ No newline at end of file > +<content>{{include document="IRC.IRCArchiveClassSheet"/}}</content> > +</xwikidoc> > > Modified: platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCBot.xml > =================================================================== > --- platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCBot.xml 2010-12-15 19:57:51 UTC (rev 33547) > +++ platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/IRCBot.xml 2010-12-15 19:58:05 UTC (rev 33548) > @@ -188,8 +188,8 @@ > * [[$item]]: $itemDoc.getValue("description") > #end > {{/velocity}} > -))) > > + > = Status = > > {{include document="IRC.IRCBotlet"/}} > @@ -258,8 +258,14 @@ > { > initExecutionContext() > def date = xwiki.formatDate(new Date(), "HH:mm", context) > - log(channel, "${sender} - (${date}): ${message}\n") > - > + def User[] pircUsers = this.getUsers(channel) > + def userPrefix = ""; > + for (User singleUser in pircUsers) { > + if (singleUser.equals(sender)) { > + userPrefix = singleUser.getPrefix(); > + } > + } > + log(channel, "${date} <${userPrefix}${sender}> ${message}\n") > for (IRCBotlet botlet : Utils.getComponentManager().lookupList(IRCBotlet.class)) { > try { > def result = botlet.handleMessage(channel, sender, login, hostname, message, xwiki, context) > @@ -291,21 +297,21 @@ > { > initExecutionContext() > def date = xwiki.formatDate(new Date(), "HH:mm", context) > - log(channel, "${sender} joined ${channel} at ${date}\n") > + log(channel, "${date} <${sender}> has joined ${channel}\n") > } > > void onPart(String channel, String sender, String login, String hostname) > { > initExecutionContext() > def date = xwiki.formatDate(new Date(), "HH:mm", context) > - log(channel, "${sender} left ${channel} at ${date}\n") > + log(channel, "${date} <${sender}> has left ${channel}\n") > } > > void onQuit(String sourceNick, String sourceLogin, String sourceHostname, String reason) > { > initExecutionContext() > def date = xwiki.formatDate(new Date(), "HH:mm", context) > - log(getChannels()[0], "${sourceNick} left at ${date} (${reason}\n") > + log(getChannels()[0], "${date} <${sourceNick}> has quit\n") > } > > public void onPrivateMessage(String sender, String login, String hostname, String message) > @@ -317,7 +323,8 @@ > void onNickChange(String oldNick, String login, String hostname, String newNick) > { > initExecutionContext() > - log(getChannels()[0], "${oldNick} is now known as ${newNick} (${login}@${hostname}\n") > + def date = xwiki.formatDate(new Date(), "HH:mm", context) > + log(getChannels()[0], "${date} <${oldNick}> is now known as <${newNick}>\n") > } > > XWikiDocument getLoggingDocument(String channelName) > > Modified: platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/WebHome.xml > =================================================================== > --- platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/WebHome.xml 2010-12-15 19:57:51 UTC (rev 33547) > +++ platform/xwiki-applications/trunk/ircbot/src/main/resources/IRC/WebHome.xml 2010-12-15 19:58:05 UTC (rev 33548) > @@ -37,6 +37,7 @@ > }) > #set($options = { > "className":"IRC.IRCArchiveClass", > + "translationPrefix" : "irc.", > "rowCount": 10 > }) > #livetable("ircarchives" $columns $columnsProperties $options)

14 years, 9 months

1
0
0 0

Re: [xwiki-devs] [xwiki-notifications] r33381 - platform/core/trunk/xwiki-rendering/xwiki-rendering-tests/src/test/resources/simple/encoding

by Vincent Massol

We need to be careful about this. This test proves that you've changed the behavior. And changing the behavior is an important change. In this case Thomas and I had designed the XHTML renderer so that its output would work with any encoding. I think this is no longer true with your change in http://jira.xwiki.org/jira/browse/XWIKI-5763 To be honest, I'm a bit worried about that change unless you really know what you are doing... especially in XE 2.7 which is the last minor release of the 2.x cycle. Thanks -Vincent On Dec 14, 2010, at 4:23 PM, sdumitriu (SVN) wrote: > Author: sdumitriu > Date: 2010-12-14 16:23:00 +0100 (Tue, 14 Dec 2010) > New Revision: 33381 > > Modified: > platform/core/trunk/xwiki-rendering/xwiki-rendering-tests/src/test/resources/simple/encoding/encoding1.test > Log: > [misc] Disable a failing test until we decide what should be done with non-ASCII characters > > Modified: platform/core/trunk/xwiki-rendering/xwiki-rendering-tests/src/test/resources/simple/encoding/encoding1.test > =================================================================== > --- platform/core/trunk/xwiki-rendering/xwiki-rendering-tests/src/test/resources/simple/encoding/encoding1.test 2010-12-14 14:51:26 UTC (rev 33380) > +++ platform/core/trunk/xwiki-rendering/xwiki-rendering-tests/src/test/resources/simple/encoding/encoding1.test 2010-12-14 15:23:00 UTC (rev 33381) > @@ -18,6 +18,7 @@ > endParagraph [[param1]=[val(((252)))e]] > endDocument > .#----------------------------------------------------- > -.expect|xhtml/1.0 > +.expect|xhtml/1.0|skip > +.# TODO: Re-enable this test once we reach a decision about non-ASCII characters > .#----------------------------------------------------- >  ˜€ ˜€

14 years, 9 months

1
0
0 0

Re: [xwiki-devs] [xwiki-notifications] r33376 - in platform/core/trunk: xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge/event xwiki-core/src/main/java/com/xpn/xwiki/internal/cache xwiki-core/src/main/java/com/xpn/xwiki/internal/event xwiki-l

by Thomas Mortagne

On Tue, Dec 14, 2010 at 15:27, Jerome Velociter <jerome(a)xwiki.com> wrote: > Thanks... > > I think you've forgot to update other events that rely on > AbstractDocumentEvent. if I remember well there is one in > xwiki-annotations-core, and maybe some in xwiki-core too. I did updated xwiki-core ones but forgot some others yes. > > On Tue, Dec 14, 2010 at 3:01 PM, tmortagne > <platform-notifications(a)xwiki.org> wrote: >> Author: tmortagne >> Date: 2010-12-14 15:01:28 +0100 (Tue, 14 Dec 2010) >> New Revision: 33376 >> >> Added: >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/ >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/ >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java >> platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/legacy/ >> platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/legacy/internal/ >> platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java >> Removed: >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java >> platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java >> platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java >> Modified: >> platform/core/trunk/xwiki-bridge/pom.xml >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java >> platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java >> platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java >> platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java >> platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java >> platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java >> platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt >> Log: >> XWIKI-5786: Deprecate Document(Save|Update|Delete)Event and add new Document(Created|Updated|Deleted)Event in xwiki-bridge instead >> * LegacyEventDispatcher loses document name >> * LegacyEventDispatcher should be in an internal package >> * AbstractDocumentEvent has nothing to do in xwiki-model >> >> Modified: platform/core/trunk/xwiki-bridge/pom.xml >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/pom.xml 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/pom.xml 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -56,13 +56,6 @@ >>  >> <groupId>org.apache.maven.plugins</groupId> >> <artifactId>maven-checkstyle-plugin</artifactId> >> - <dependencies> >> - <dependency> >> - <groupId>org.xwiki.platform.tools</groupId> >> - <artifactId>xwiki-verification-resources</artifactId> >> - <version>${platform.tool.verification.version}</version> >> - </dependency> >> - </dependencies> >> </plugin> >> <plugin> >> <groupId>org.codehaus.mojo</groupId> >> >> Copied: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java (from rev 33375, platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java) >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java (rev 0) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -0,0 +1,74 @@ >> +/* >> + * 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.bridge.event; >> + >> +import org.xwiki.model.internal.reference.DefaultStringEntityReferenceSerializer; >> +import org.xwiki.model.reference.DocumentReference; >> +import org.xwiki.observation.event.AbstractFilterableEvent; >> +import org.xwiki.observation.event.filter.EventFilter; >> + >> +/** >> + * Base class for all document related {@link org.xwiki.observation.event.Event}. >> + * >> + * @version $Id$ >> + * @since 2.7RC1 >> + */ >> +public abstract class AbstractDocumentEvent extends AbstractFilterableEvent >> +{ >> + /** >> + * The version identifier for this Serializable class. Increment only if the serialized form of the class >> + * changes. >> + */ >> + private static final long serialVersionUID = 1L; >> + >> + /** >> + * Used to serialize document name. >> + */ >> + private static final DefaultStringEntityReferenceSerializer SERIALIZER = >> + new DefaultStringEntityReferenceSerializer(); >> + >> + /** >> + * This event will match any other document event of the same type. >> + */ >> + public AbstractDocumentEvent() >> + { >> + super(); >> + } >> + >> + /** >> + * This event will match only events of the same type affecting the same document. >> + * >> + * @param documentReference the reference of the document relater to this event >> + */ >> + public AbstractDocumentEvent(DocumentReference documentReference) >> + { >> + super(SERIALIZER.serialize(documentReference)); >> + } >> + >> + /** >> + * Constructor using a custom {@link EventFilter}. >> + * >> + * @param eventFilter the filter to use for matching events >> + */ >> + public AbstractDocumentEvent(EventFilter eventFilter) >> + { >> + super(eventFilter); >> + } >> +} >> >> Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,13 +20,13 @@ >> package org.xwiki.bridge.event; >> >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> >> /** >> * An event triggered after a document is created. >> * >> * @version $Id$ >> + * @since 2.7RC1 >> */ >> public class DocumentCreatedEvent extends AbstractDocumentEvent >> { >> >> Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,13 +20,13 @@ >> package org.xwiki.bridge.event; >> >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> >> /** >> * An event triggered before a document is created. >> * >> * @version $Id$ >> + * @since 2.7RC1 >> */ >> public class DocumentCreatingEvent extends AbstractDocumentEvent >> { >> >> Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,13 +20,13 @@ >> package org.xwiki.bridge.event; >> >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> >> /** >> * An event triggered after a document is deleted. >> * >> * @version $Id$ >> + * @since 2.7RC1 >> */ >> public class DocumentDeletedEvent extends AbstractDocumentEvent >> { >> @@ -43,7 +43,7 @@ >> */ >> public DocumentDeletedEvent() >> { >> - super(); >> + >> } >> >> /** >> >> Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,13 +20,13 @@ >> package org.xwiki.bridge.event; >> >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> >> /** >> * An event triggered before a document is deleted. >> * >> * @version $Id$ >> + * @since 2.7RC1 >> */ >> public class DocumentDeletingEvent extends AbstractDocumentEvent >> { >> >> Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,13 +20,13 @@ >> package org.xwiki.bridge.event; >> >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> >> /** >> * An event triggered after a document is updated. >> * >> * @version $Id$ >> + * @since 2.7RC1 >> */ >> public class DocumentUpdatedEvent extends AbstractDocumentEvent >> { >> >> Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,13 +20,13 @@ >> package org.xwiki.bridge.event; >> >> import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> >> /** >> * An event triggered before a document is updated. >> * >> * @version $Id$ >> + * @since 2.7RC1 >> */ >> public class DocumentUpdatingEvent extends AbstractDocumentEvent >> { >> @@ -35,7 +35,7 @@ >> * changes. >> */ >> private static final long serialVersionUID = 1L; >> - >> + >> /** >> * Constructor initializing the event filter with an >> * {@link org.xwiki.observation.event.filter.AlwaysMatchingEventFilter}, meaning that this event will match any >> >> Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java >> =================================================================== >> --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -24,6 +24,7 @@ >> import java.util.HashSet; >> import java.util.List; >> >> +import org.xwiki.bridge.event.AbstractDocumentEvent; >> import org.xwiki.bridge.event.DocumentCreatedEvent; >> import org.xwiki.bridge.event.DocumentDeletedEvent; >> import org.xwiki.bridge.event.DocumentUpdatedEvent; >> @@ -39,7 +40,6 @@ >> import org.xwiki.model.reference.EntityReferenceSerializer; >> import org.xwiki.observation.EventListener; >> import org.xwiki.observation.ObservationManager; >> -import org.xwiki.model.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.Event; >> >> /** >> >> Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,7 +20,7 @@ >> */ >> package com.xpn.xwiki.internal.event; >> >> -import org.xwiki.model.event.AbstractDocumentEvent; >> +import org.xwiki.bridge.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> import org.xwiki.observation.event.filter.FixedNameEventFilter; >> >> >> Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -20,7 +20,7 @@ >> */ >> package com.xpn.xwiki.internal.event; >> >> -import org.xwiki.model.event.AbstractDocumentEvent; >> +import org.xwiki.bridge.event.AbstractDocumentEvent; >> import org.xwiki.observation.event.filter.EventFilter; >> import org.xwiki.observation.event.filter.FixedNameEventFilter; >> >> >> Copied: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java (from rev 33375, platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java) >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java (rev 0) >> +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -0,0 +1,120 @@ >> +/* >> + * 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.legacy.internal; >> + >> +import java.util.ArrayList; >> +import java.util.List; >> + >> +import org.xwiki.bridge.event.DocumentCreatedEvent; >> +import org.xwiki.bridge.event.DocumentDeletedEvent; >> +import org.xwiki.bridge.event.DocumentUpdatedEvent; >> +import org.xwiki.component.annotation.Component; >> +import org.xwiki.component.annotation.Requirement; >> +import org.xwiki.component.logging.AbstractLogEnabled; >> +import org.xwiki.component.manager.ComponentLookupException; >> +import org.xwiki.component.manager.ComponentManager; >> +import org.xwiki.observation.EventListener; >> +import org.xwiki.observation.ObservationManager; >> +import org.xwiki.observation.event.DocumentDeleteEvent; >> +import org.xwiki.observation.event.DocumentSaveEvent; >> +import org.xwiki.observation.event.DocumentUpdateEvent; >> +import org.xwiki.observation.event.Event; >> + >> +/** >> + * An event listener that forwards received events to their corresponding legacy events. This allows depreciated events >> + * to continue be supported. >> + * >> + * @version $Id$ >> + */ >> +@Component("legacyEventDispatcher") >> +public class LegacyEventDispatcher extends AbstractLogEnabled implements EventListener >> +{ >> + /** >> + * Component manager, used to get access to the observation manager that we cannot get injected because of a cyclic >> + * dependency. >> + */ >> + @Requirement >> + private ComponentManager componentManager; >> + >> + /** >> + * Observation manager, used to notify legacy events. >> + */ >> + private ObservationManager observationManager; >> + >> + /** >> + * {@inheritDoc} >> + */ >> + public String getName() >> + { >> + return "legacyEventDispatcher"; >> + } >> + >> + /** >> + * {@inheritDoc} >> + */ >> + public List<Event> getEvents() >> + { >> + return new ArrayList<Event>() >> + { >> + { >> + add(new DocumentDeletedEvent()); >> + add(new DocumentCreatedEvent()); >> + add(new DocumentUpdatedEvent()); >> + } >> + }; >> + } >> + >> + /** >> + * {@inheritDoc} >> + */ >> + public void onEvent(Event event, Object source, Object data) >> + { >> + if (event instanceof DocumentDeletedEvent) { >> + this.getObservationManager().notify( >> + new DocumentDeleteEvent(((DocumentDeletedEvent) event).getEventFilter()), source, data); >> + } else if (event instanceof DocumentCreatedEvent) { >> + this.getObservationManager().notify(new DocumentSaveEvent(((DocumentCreatedEvent) event).getEventFilter()), >> + source, data); >> + } else if (event instanceof DocumentUpdatedEvent) { >> + this.getObservationManager().notify( >> + new DocumentUpdateEvent(((DocumentUpdatedEvent) event).getEventFilter()), source, data); >> + } >> + } >> + >> + /** >> + * Helper to lazily lookup the observation manager. >> + * >> + * @return the observation manager >> + */ >> + private ObservationManager getObservationManager() >> + { >> + if (this.observationManager != null) { >> + return this.observationManager; >> + } >> + try { >> + this.observationManager = this.componentManager.lookup(ObservationManager.class); >> + } catch (ComponentLookupException e) { >> + this.getLogger().error("Failed to lookup observation manager", e); >> + } >> + return this.observationManager; >> + } >> + >> +} >> >> Deleted: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -1,115 +0,0 @@ >> -/* >> - * 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.observation; >> - >> -import java.util.ArrayList; >> -import java.util.List; >> - >> -import org.xwiki.bridge.event.DocumentCreatedEvent; >> -import org.xwiki.bridge.event.DocumentDeletedEvent; >> -import org.xwiki.bridge.event.DocumentUpdatedEvent; >> -import org.xwiki.component.annotation.Component; >> -import org.xwiki.component.annotation.Requirement; >> -import org.xwiki.component.logging.AbstractLogEnabled; >> -import org.xwiki.component.manager.ComponentLookupException; >> -import org.xwiki.component.manager.ComponentManager; >> -import org.xwiki.observation.event.DocumentDeleteEvent; >> -import org.xwiki.observation.event.DocumentSaveEvent; >> -import org.xwiki.observation.event.DocumentUpdateEvent; >> -import org.xwiki.observation.event.Event; >> - >> -/** >> - * An event listener that forwards received events to their corresponding legacy events. This allows depreciated events >> - * to continue be supported. >> - * >> - * @version $Id$ >> - */ >> -@Component("legacyEventDispatcher") >> -public class LegacyEventDispatcher extends AbstractLogEnabled implements EventListener >> -{ >> - /** >> - * Component manager, used to get access to the observation manager that we cannot get injected because of a cyclic >> - * dependency. >> - */ >> - @Requirement >> - private ComponentManager componentManager; >> - >> - /** >> - * Observation manager, used to notify legacy events. >> - */ >> - private ObservationManager observationManager; >> - >> - /** >> - * {@inheritDoc} >> - */ >> - public String getName() >> - { >> - return "legacyEventDispatcher"; >> - } >> - >> - /** >> - * {@inheritDoc} >> - */ >> - public List<Event> getEvents() >> - { >> - return new ArrayList<Event>() >> - { >> - { >> - add(new DocumentDeletedEvent()); >> - add(new DocumentCreatedEvent()); >> - add(new DocumentUpdatedEvent()); >> - } >> - }; >> - } >> - >> - /** >> - * {@inheritDoc} >> - */ >> - public void onEvent(Event event, Object source, Object data) >> - { >> - if (event instanceof DocumentDeletedEvent) { >> - this.getObservationManager().notify(new DocumentDeleteEvent(), source, data); >> - } else if (event instanceof DocumentCreatedEvent) { >> - this.getObservationManager().notify(new DocumentSaveEvent(), source, data); >> - } else if (event instanceof DocumentUpdatedEvent) { >> - this.getObservationManager().notify(new DocumentUpdateEvent(), source, data); >> - } >> - } >> - >> - /** >> - * Helper to lazily lookup the observation manager. >> - * >> - * @return the observation manager >> - */ >> - private ObservationManager getObservationManager() >> - { >> - if (this.observationManager != null) { >> - return this.observationManager; >> - } >> - try { >> - this.observationManager = this.componentManager.lookup(ObservationManager.class); >> - } catch (ComponentLookupException e) { >> - this.getLogger().error("Failed to lookup observation manager", e); >> - } >> - return this.observationManager; >> - } >> - >> -} >> >> Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -27,7 +27,9 @@ >> * Base class for all document {@link Event events}. >> * >> * @version $Id$ >> + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.AbstractDocumentEvent} instead >> */ >> +@Deprecated >> public abstract class AbstractDocumentEvent extends AbstractFilterableEvent >> { >> /** >> @@ -41,7 +43,7 @@ >> */ >> public AbstractDocumentEvent() >> { >> - super(); >> + super(); >> } >> >> /** >> >> Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -26,7 +26,7 @@ >> * An event triggered when a document is deleted. >> * >> * @version $Id$ >> - * @deprecated use {@link org.xwiki.bridge.event.DocumentDeletedEvent} instead >> + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.DocumentDeletedEvent} instead >> */ >> @Deprecated >> public class DocumentDeleteEvent extends AbstractDocumentEvent >> >> Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -26,7 +26,7 @@ >> * An event triggered when a document is saved for the first time (a new document is created). >> * >> * @version $Id$ >> - * @deprecated use {@link org.xwiki.bridge.event.DocumentCreatedEvent} instead >> + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.DocumentCreatedEvent} instead >> */ >> @Deprecated >> public class DocumentSaveEvent extends AbstractDocumentEvent >> >> Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -26,7 +26,7 @@ >> * An event triggered when an existing document is saved (updated). >> * >> * @version $Id$ >> - * @deprecated use {@link org.xwiki.bridge.event.DocumentUpdatedEvent} instead >> + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.DocumentUpdatedEvent} instead >> */ >> @Deprecated >> public class DocumentUpdateEvent extends AbstractDocumentEvent >> >> Modified: platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -1 +1 @@ >> -org.xwiki.observation.LegacyEventDispatcher >> +org.xwiki.legacy.internal.LegacyEventDispatcher >> >> Deleted: platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -1,108 +0,0 @@ >> -/* >> - * 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.observation; >> - >> -import java.util.Collections; >> -import java.util.List; >> - >> -import junit.framework.Assert; >> - >> -import org.junit.Before; >> -import org.junit.Test; >> -import org.xwiki.bridge.event.DocumentCreatedEvent; >> -import org.xwiki.bridge.event.DocumentDeletedEvent; >> -import org.xwiki.bridge.event.DocumentUpdatedEvent; >> -import org.xwiki.observation.event.DocumentDeleteEvent; >> -import org.xwiki.observation.event.DocumentSaveEvent; >> -import org.xwiki.observation.event.DocumentUpdateEvent; >> -import org.xwiki.observation.event.Event; >> -import org.xwiki.test.AbstractComponentTestCase; >> - >> -public class LegacyEventDispatcherTest extends AbstractComponentTestCase >> -{ >> - >> - private boolean hasBeenNotified; >> - >> - private ObservationManager om; >> - >> - @Before >> - public void setUp() throws Exception >> - { >> - this.hasBeenNotified = false; >> - this.om = this.getComponentManager().lookup(ObservationManager.class); >> - } >> - >> - @Test >> - public void testLegacyDocumentDeleteEventGetsDispatched() throws Exception >> - { >> - this.registerListenerWithLegacyEvent(new DocumentDeleteEvent()); >> - om.notify(new DocumentDeletedEvent(), new Object()); >> - // The notification is synchronous, so the following assertion will only be tested >> - // once all matching event listeners have been notified. >> - Assert.assertTrue("Should have been notified by legacy event dispatcher", this.hasBeenNotified); >> - } >> - >> - @Test >> - public void testLegacyDocumentSaveEventGetsDispatched() throws Exception >> - { >> - this.registerListenerWithLegacyEvent(new DocumentSaveEvent()); >> - om.notify(new DocumentCreatedEvent(), new Object()); >> - // The notification is synchronous, so the following assertion will only be tested >> - // once all matching event listeners have been notified. >> - Assert.assertTrue("Should have been notified by legacy event dispatcher", this.hasBeenNotified); >> - } >> - >> - @Test >> - public void testLegacyDocumentUpdateEventGetsDispatched() throws Exception >> - { >> - this.registerListenerWithLegacyEvent(new DocumentUpdateEvent()); >> - om.notify(new DocumentUpdatedEvent(), new Object()); >> - // The notification is synchronous, so the following assertion will only be tested >> - // once all matching event listeners have been notified. >> - Assert.assertTrue("Should have been notified by legacy event dispatcher", this.hasBeenNotified); >> - } >> - >> - private void registerListenerWithLegacyEvent(final Event event) >> - { >> - this.om.addListener(new EventListener() >> - { >> - public String getName() >> - { >> - return "testLegacyEventDispatchEventListener"; >> - } >> - >> - public List<Event> getEvents() >> - { >> - return Collections.<Event> singletonList(event); >> - } >> - >> - public void onEvent(Event event, Object source, Object data) >> - { >> - setNotified(); >> - } >> - >> - }); >> - } >> - >> - private void setNotified() >> - { >> - this.hasBeenNotified = true; >> - } >> -} >> >> Copied: platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java (from rev 33375, platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java) >> =================================================================== >> --- platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java (rev 0) >> +++ platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -0,0 +1,111 @@ >> +/* >> + * 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.observation.event; >> + >> +import java.util.Collections; >> +import java.util.List; >> + >> +import junit.framework.Assert; >> + >> +import org.junit.Before; >> +import org.junit.Test; >> +import org.xwiki.bridge.event.DocumentCreatedEvent; >> +import org.xwiki.bridge.event.DocumentDeletedEvent; >> +import org.xwiki.bridge.event.DocumentUpdatedEvent; >> +import org.xwiki.model.reference.DocumentReference; >> +import org.xwiki.observation.EventListener; >> +import org.xwiki.observation.ObservationManager; >> +import org.xwiki.observation.event.DocumentDeleteEvent; >> +import org.xwiki.observation.event.DocumentSaveEvent; >> +import org.xwiki.observation.event.DocumentUpdateEvent; >> +import org.xwiki.observation.event.Event; >> +import org.xwiki.observation.event.filter.EventFilter; >> +import org.xwiki.observation.event.filter.FixedNameEventFilter; >> +import org.xwiki.test.AbstractComponentTestCase; >> + >> +public class LegacyEventDispatcherTest extends AbstractComponentTestCase >> +{ >> + private Event receivedEvent; >> + >> + private ObservationManager om; >> + >> + @Before >> + public void setUp() throws Exception >> + { >> + this.om = getComponentManager().lookup(ObservationManager.class); >> + } >> + >> + @Test >> + public void testLegacyDocumentDeleteEventGetsDispatched() throws Exception >> + { >> + this.registerListenerWithLegacyEvent(new DocumentDeleteEvent()); >> + this.om.notify(new DocumentDeletedEvent(new DocumentReference("wiki", "space", "name")), new Object()); >> + >> + // The notification is synchronous, so the following assertion will only be tested >> + // once all matching event listeners have been notified. >> + Assert.assertNotNull("Should have been notified by legacy event dispatcher", this.receivedEvent); >> + Assert.assertEquals("Wrong event filter", "wiki:space.name", ((EventFilter)((FilterableEvent)this.receivedEvent).getEventFilter()).getFilter()); >> + } >> + >> + @Test >> + public void testLegacyDocumentSaveEventGetsDispatched() throws Exception >> + { >> + this.registerListenerWithLegacyEvent(new DocumentSaveEvent()); >> + this.om.notify(new DocumentCreatedEvent(new DocumentReference("wiki", "space", "name")), new Object()); >> + >> + // The notification is synchronous, so the following assertion will only be tested >> + // once all matching event listeners have been notified. >> + Assert.assertNotNull("Should have been notified by legacy event dispatcher", this.receivedEvent); >> + Assert.assertEquals("Wrong event filter", "wiki:space.name", ((EventFilter)((FilterableEvent)this.receivedEvent).getEventFilter()).getFilter()); >> + } >> + >> + @Test >> + public void testLegacyDocumentUpdateEventGetsDispatched() throws Exception >> + { >> + this.registerListenerWithLegacyEvent(new DocumentUpdateEvent()); >> + this.om.notify(new DocumentUpdatedEvent(new DocumentReference("wiki", "space", "name")), new Object()); >> + >> + // The notification is synchronous, so the following assertion will only be tested >> + // once all matching event listeners have been notified. >> + Assert.assertNotNull("Should have been notified by legacy event dispatcher", this.receivedEvent); >> + Assert.assertEquals("Wrong event filter", "wiki:space.name", ((EventFilter)((FilterableEvent)this.receivedEvent).getEventFilter()).getFilter()); >> + } >> + >> + private void registerListenerWithLegacyEvent(final Event event) >> + { >> + this.om.addListener(new EventListener() >> + { >> + public String getName() >> + { >> + return "testLegacyEventDispatchEventListener"; >> + } >> + >> + public List<Event> getEvents() >> + { >> + return Collections.<Event> singletonList(event); >> + } >> + >> + public void onEvent(Event event, Object source, Object data) >> + { >> + receivedEvent = event; >> + } >> + }); >> + } >> +} >> >> >> Property changes on: platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java >> ___________________________________________________________________ >> Added: svn:keywords >> + Id >> >> Deleted: platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java >> =================================================================== >> --- platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) >> +++ platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) >> @@ -1,68 +0,0 @@ >> -/* >> - * 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.model.event; >> - >> -import org.xwiki.model.internal.reference.DefaultStringEntityReferenceSerializer; >> -import org.xwiki.model.reference.DocumentReference; >> -import org.xwiki.observation.event.AbstractFilterableEvent; >> -import org.xwiki.observation.event.Event; >> -import org.xwiki.observation.event.filter.EventFilter; >> - >> -/** >> - * Base class for all document {@link Event events}. >> - * >> - * @version $Id$ >> - */ >> -public abstract class AbstractDocumentEvent extends AbstractFilterableEvent >> -{ >> - /** >> - * The version identifier for this Serializable class. Increment only if the serialized form of the class >> - * changes. >> - */ >> - private static final long serialVersionUID = 1L; >> - >> - /** >> - * This event will match any other document event of the same type. >> - */ >> - public AbstractDocumentEvent() >> - { >> - super(); >> - } >> - >> - /** >> - * This event will match only events of the same type affecting the same document. >> - * >> - * @param documentReference the reference of the document relater to this event >> - */ >> - public AbstractDocumentEvent(DocumentReference documentReference) >> - { >> - super(new DefaultStringEntityReferenceSerializer().serialize(documentReference)); >> - } >> - >> - /** >> - * Constructor using a custom {@link EventFilter}. >> - * >> - * @param eventFilter the filter to use for matching events >> - */ >> - public AbstractDocumentEvent(EventFilter eventFilter) >> - { >> - super(eventFilter); >> - } >> -} >> \ No newline at end of file >> >> _______________________________________________ >> notifications mailing list >> notifications(a)xwiki.org >> http://lists.xwiki.org/mailman/listinfo/notifications >> > _______________________________________________ > devs mailing list > devs(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/devs > -- Thomas Mortagne

14 years, 9 months

1
0
0 0

Re: [xwiki-devs] [xwiki-notifications] r33376 - in platform/core/trunk: xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge/event xwiki-core/src/main/java/com/xpn/xwiki/internal/cache xwiki-core/src/main/java/com/xpn/xwiki/internal/event xwiki-legacy/src/mai

by Jerome Velociter

Thanks... I think you've forgot to update other events that rely on AbstractDocumentEvent. if I remember well there is one in xwiki-annotations-core, and maybe some in xwiki-core too. On Tue, Dec 14, 2010 at 3:01 PM, tmortagne <platform-notifications(a)xwiki.org> wrote: > Author: tmortagne > Date: 2010-12-14 15:01:28 +0100 (Tue, 14 Dec 2010) > New Revision: 33376 > > Added: > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/ > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/ > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/legacy/ > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/legacy/internal/ > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java > Removed: > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java > platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java > Modified: > platform/core/trunk/xwiki-bridge/pom.xml > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt > Log: > XWIKI-5786: Deprecate Document(Save|Update|Delete)Event and add new Document(Created|Updated|Deleted)Event in xwiki-bridge instead > * LegacyEventDispatcher loses document name > * LegacyEventDispatcher should be in an internal package > * AbstractDocumentEvent has nothing to do in xwiki-model > > Modified: platform/core/trunk/xwiki-bridge/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-bridge/pom.xml 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/pom.xml 2010-12-14 14:01:28 UTC (rev 33376) > @@ -56,13 +56,6 @@ >  > <groupId>org.apache.maven.plugins</groupId> > <artifactId>maven-checkstyle-plugin</artifactId> > - <dependencies> > - <dependency> > - <groupId>org.xwiki.platform.tools</groupId> > - <artifactId>xwiki-verification-resources</artifactId> > - <version>${platform.tool.verification.version}</version> > - </dependency> > - </dependencies> > </plugin> > <plugin> > <groupId>org.codehaus.mojo</groupId> > > Copied: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java (from rev 33375, platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java) > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java (rev 0) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/AbstractDocumentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -0,0 +1,74 @@ > +/* > + * 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.bridge.event; > + > +import org.xwiki.model.internal.reference.DefaultStringEntityReferenceSerializer; > +import org.xwiki.model.reference.DocumentReference; > +import org.xwiki.observation.event.AbstractFilterableEvent; > +import org.xwiki.observation.event.filter.EventFilter; > + > +/** > + * Base class for all document related {@link org.xwiki.observation.event.Event}. > + * > + * @version $Id$ > + * @since 2.7RC1 > + */ > +public abstract class AbstractDocumentEvent extends AbstractFilterableEvent > +{ > + /** > + * The version identifier for this Serializable class. Increment only if the serialized form of the class > + * changes. > + */ > + private static final long serialVersionUID = 1L; > + > + /** > + * Used to serialize document name. > + */ > + private static final DefaultStringEntityReferenceSerializer SERIALIZER = > + new DefaultStringEntityReferenceSerializer(); > + > + /** > + * This event will match any other document event of the same type. > + */ > + public AbstractDocumentEvent() > + { > + super(); > + } > + > + /** > + * This event will match only events of the same type affecting the same document. > + * > + * @param documentReference the reference of the document relater to this event > + */ > + public AbstractDocumentEvent(DocumentReference documentReference) > + { > + super(SERIALIZER.serialize(documentReference)); > + } > + > + /** > + * Constructor using a custom {@link EventFilter}. > + * > + * @param eventFilter the filter to use for matching events > + */ > + public AbstractDocumentEvent(EventFilter eventFilter) > + { > + super(eventFilter); > + } > +} > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,13 +20,13 @@ > package org.xwiki.bridge.event; > > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > > /** > * An event triggered after a document is created. > * > * @version $Id$ > + * @since 2.7RC1 > */ > public class DocumentCreatedEvent extends AbstractDocumentEvent > { > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,13 +20,13 @@ > package org.xwiki.bridge.event; > > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > > /** > * An event triggered before a document is created. > * > * @version $Id$ > + * @since 2.7RC1 > */ > public class DocumentCreatingEvent extends AbstractDocumentEvent > { > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,13 +20,13 @@ > package org.xwiki.bridge.event; > > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > > /** > * An event triggered after a document is deleted. > * > * @version $Id$ > + * @since 2.7RC1 > */ > public class DocumentDeletedEvent extends AbstractDocumentEvent > { > @@ -43,7 +43,7 @@ > */ > public DocumentDeletedEvent() > { > - super(); > + > } > > /** > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,13 +20,13 @@ > package org.xwiki.bridge.event; > > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > > /** > * An event triggered before a document is deleted. > * > * @version $Id$ > + * @since 2.7RC1 > */ > public class DocumentDeletingEvent extends AbstractDocumentEvent > { > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,13 +20,13 @@ > package org.xwiki.bridge.event; > > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > > /** > * An event triggered after a document is updated. > * > * @version $Id$ > + * @since 2.7RC1 > */ > public class DocumentUpdatedEvent extends AbstractDocumentEvent > { > > Modified: platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java > =================================================================== > --- platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,13 +20,13 @@ > package org.xwiki.bridge.event; > > import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > > /** > * An event triggered before a document is updated. > * > * @version $Id$ > + * @since 2.7RC1 > */ > public class DocumentUpdatingEvent extends AbstractDocumentEvent > { > @@ -35,7 +35,7 @@ > * changes. > */ > private static final long serialVersionUID = 1L; > - > + > /** > * Constructor initializing the event filter with an > * {@link org.xwiki.observation.event.filter.AlwaysMatchingEventFilter}, meaning that this event will match any > > Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java > =================================================================== > --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -24,6 +24,7 @@ > import java.util.HashSet; > import java.util.List; > > +import org.xwiki.bridge.event.AbstractDocumentEvent; > import org.xwiki.bridge.event.DocumentCreatedEvent; > import org.xwiki.bridge.event.DocumentDeletedEvent; > import org.xwiki.bridge.event.DocumentUpdatedEvent; > @@ -39,7 +40,6 @@ > import org.xwiki.model.reference.EntityReferenceSerializer; > import org.xwiki.observation.EventListener; > import org.xwiki.observation.ObservationManager; > -import org.xwiki.model.event.AbstractDocumentEvent; > import org.xwiki.observation.event.Event; > > /** > > Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java > =================================================================== > --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractAttachmentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,7 +20,7 @@ > */ > package com.xpn.xwiki.internal.event; > > -import org.xwiki.model.event.AbstractDocumentEvent; > +import org.xwiki.bridge.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > import org.xwiki.observation.event.filter.FixedNameEventFilter; > > > Modified: platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java > =================================================================== > --- platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/event/AbstractCommentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -20,7 +20,7 @@ > */ > package com.xpn.xwiki.internal.event; > > -import org.xwiki.model.event.AbstractDocumentEvent; > +import org.xwiki.bridge.event.AbstractDocumentEvent; > import org.xwiki.observation.event.filter.EventFilter; > import org.xwiki.observation.event.filter.FixedNameEventFilter; > > > Copied: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java (from rev 33375, platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java) > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java (rev 0) > +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/legacy/internal/LegacyEventDispatcher.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -0,0 +1,120 @@ > +/* > + * 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.legacy.internal; > + > +import java.util.ArrayList; > +import java.util.List; > + > +import org.xwiki.bridge.event.DocumentCreatedEvent; > +import org.xwiki.bridge.event.DocumentDeletedEvent; > +import org.xwiki.bridge.event.DocumentUpdatedEvent; > +import org.xwiki.component.annotation.Component; > +import org.xwiki.component.annotation.Requirement; > +import org.xwiki.component.logging.AbstractLogEnabled; > +import org.xwiki.component.manager.ComponentLookupException; > +import org.xwiki.component.manager.ComponentManager; > +import org.xwiki.observation.EventListener; > +import org.xwiki.observation.ObservationManager; > +import org.xwiki.observation.event.DocumentDeleteEvent; > +import org.xwiki.observation.event.DocumentSaveEvent; > +import org.xwiki.observation.event.DocumentUpdateEvent; > +import org.xwiki.observation.event.Event; > + > +/** > + * An event listener that forwards received events to their corresponding legacy events. This allows depreciated events > + * to continue be supported. > + * > + * @version $Id$ > + */ > +@Component("legacyEventDispatcher") > +public class LegacyEventDispatcher extends AbstractLogEnabled implements EventListener > +{ > + /** > + * Component manager, used to get access to the observation manager that we cannot get injected because of a cyclic > + * dependency. > + */ > + @Requirement > + private ComponentManager componentManager; > + > + /** > + * Observation manager, used to notify legacy events. > + */ > + private ObservationManager observationManager; > + > + /** > + * {@inheritDoc} > + */ > + public String getName() > + { > + return "legacyEventDispatcher"; > + } > + > + /** > + * {@inheritDoc} > + */ > + public List<Event> getEvents() > + { > + return new ArrayList<Event>() > + { > + { > + add(new DocumentDeletedEvent()); > + add(new DocumentCreatedEvent()); > + add(new DocumentUpdatedEvent()); > + } > + }; > + } > + > + /** > + * {@inheritDoc} > + */ > + public void onEvent(Event event, Object source, Object data) > + { > + if (event instanceof DocumentDeletedEvent) { > + this.getObservationManager().notify( > + new DocumentDeleteEvent(((DocumentDeletedEvent) event).getEventFilter()), source, data); > + } else if (event instanceof DocumentCreatedEvent) { > + this.getObservationManager().notify(new DocumentSaveEvent(((DocumentCreatedEvent) event).getEventFilter()), > + source, data); > + } else if (event instanceof DocumentUpdatedEvent) { > + this.getObservationManager().notify( > + new DocumentUpdateEvent(((DocumentUpdatedEvent) event).getEventFilter()), source, data); > + } > + } > + > + /** > + * Helper to lazily lookup the observation manager. > + * > + * @return the observation manager > + */ > + private ObservationManager getObservationManager() > + { > + if (this.observationManager != null) { > + return this.observationManager; > + } > + try { > + this.observationManager = this.componentManager.lookup(ObservationManager.class); > + } catch (ComponentLookupException e) { > + this.getLogger().error("Failed to lookup observation manager", e); > + } > + return this.observationManager; > + } > + > +} > > Deleted: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -1,115 +0,0 @@ > -/* > - * 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.observation; > - > -import java.util.ArrayList; > -import java.util.List; > - > -import org.xwiki.bridge.event.DocumentCreatedEvent; > -import org.xwiki.bridge.event.DocumentDeletedEvent; > -import org.xwiki.bridge.event.DocumentUpdatedEvent; > -import org.xwiki.component.annotation.Component; > -import org.xwiki.component.annotation.Requirement; > -import org.xwiki.component.logging.AbstractLogEnabled; > -import org.xwiki.component.manager.ComponentLookupException; > -import org.xwiki.component.manager.ComponentManager; > -import org.xwiki.observation.event.DocumentDeleteEvent; > -import org.xwiki.observation.event.DocumentSaveEvent; > -import org.xwiki.observation.event.DocumentUpdateEvent; > -import org.xwiki.observation.event.Event; > - > -/** > - * An event listener that forwards received events to their corresponding legacy events. This allows depreciated events > - * to continue be supported. > - * > - * @version $Id$ > - */ > -@Component("legacyEventDispatcher") > -public class LegacyEventDispatcher extends AbstractLogEnabled implements EventListener > -{ > - /** > - * Component manager, used to get access to the observation manager that we cannot get injected because of a cyclic > - * dependency. > - */ > - @Requirement > - private ComponentManager componentManager; > - > - /** > - * Observation manager, used to notify legacy events. > - */ > - private ObservationManager observationManager; > - > - /** > - * {@inheritDoc} > - */ > - public String getName() > - { > - return "legacyEventDispatcher"; > - } > - > - /** > - * {@inheritDoc} > - */ > - public List<Event> getEvents() > - { > - return new ArrayList<Event>() > - { > - { > - add(new DocumentDeletedEvent()); > - add(new DocumentCreatedEvent()); > - add(new DocumentUpdatedEvent()); > - } > - }; > - } > - > - /** > - * {@inheritDoc} > - */ > - public void onEvent(Event event, Object source, Object data) > - { > - if (event instanceof DocumentDeletedEvent) { > - this.getObservationManager().notify(new DocumentDeleteEvent(), source, data); > - } else if (event instanceof DocumentCreatedEvent) { > - this.getObservationManager().notify(new DocumentSaveEvent(), source, data); > - } else if (event instanceof DocumentUpdatedEvent) { > - this.getObservationManager().notify(new DocumentUpdateEvent(), source, data); > - } > - } > - > - /** > - * Helper to lazily lookup the observation manager. > - * > - * @return the observation manager > - */ > - private ObservationManager getObservationManager() > - { > - if (this.observationManager != null) { > - return this.observationManager; > - } > - try { > - this.observationManager = this.componentManager.lookup(ObservationManager.class); > - } catch (ComponentLookupException e) { > - this.getLogger().error("Failed to lookup observation manager", e); > - } > - return this.observationManager; > - } > - > -} > > Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -27,7 +27,9 @@ > * Base class for all document {@link Event events}. > * > * @version $Id$ > + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.AbstractDocumentEvent} instead > */ > +@Deprecated > public abstract class AbstractDocumentEvent extends AbstractFilterableEvent > { > /** > @@ -41,7 +43,7 @@ > */ > public AbstractDocumentEvent() > { > - super(); > + super(); > } > > /** > > Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -26,7 +26,7 @@ > * An event triggered when a document is deleted. > * > * @version $Id$ > - * @deprecated use {@link org.xwiki.bridge.event.DocumentDeletedEvent} instead > + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.DocumentDeletedEvent} instead > */ > @Deprecated > public class DocumentDeleteEvent extends AbstractDocumentEvent > > Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -26,7 +26,7 @@ > * An event triggered when a document is saved for the first time (a new document is created). > * > * @version $Id$ > - * @deprecated use {@link org.xwiki.bridge.event.DocumentCreatedEvent} instead > + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.DocumentCreatedEvent} instead > */ > @Deprecated > public class DocumentSaveEvent extends AbstractDocumentEvent > > Modified: platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -26,7 +26,7 @@ > * An event triggered when an existing document is saved (updated). > * > * @version $Id$ > - * @deprecated use {@link org.xwiki.bridge.event.DocumentUpdatedEvent} instead > + * @deprecated since 2.7RC1 use {@link org.xwiki.bridge.event.DocumentUpdatedEvent} instead > */ > @Deprecated > public class DocumentUpdateEvent extends AbstractDocumentEvent > > Modified: platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt 2010-12-14 14:01:28 UTC (rev 33376) > @@ -1 +1 @@ > -org.xwiki.observation.LegacyEventDispatcher > +org.xwiki.legacy.internal.LegacyEventDispatcher > > Deleted: platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -1,108 +0,0 @@ > -/* > - * 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.observation; > - > -import java.util.Collections; > -import java.util.List; > - > -import junit.framework.Assert; > - > -import org.junit.Before; > -import org.junit.Test; > -import org.xwiki.bridge.event.DocumentCreatedEvent; > -import org.xwiki.bridge.event.DocumentDeletedEvent; > -import org.xwiki.bridge.event.DocumentUpdatedEvent; > -import org.xwiki.observation.event.DocumentDeleteEvent; > -import org.xwiki.observation.event.DocumentSaveEvent; > -import org.xwiki.observation.event.DocumentUpdateEvent; > -import org.xwiki.observation.event.Event; > -import org.xwiki.test.AbstractComponentTestCase; > - > -public class LegacyEventDispatcherTest extends AbstractComponentTestCase > -{ > - > - private boolean hasBeenNotified; > - > - private ObservationManager om; > - > - @Before > - public void setUp() throws Exception > - { > - this.hasBeenNotified = false; > - this.om = this.getComponentManager().lookup(ObservationManager.class); > - } > - > - @Test > - public void testLegacyDocumentDeleteEventGetsDispatched() throws Exception > - { > - this.registerListenerWithLegacyEvent(new DocumentDeleteEvent()); > - om.notify(new DocumentDeletedEvent(), new Object()); > - // The notification is synchronous, so the following assertion will only be tested > - // once all matching event listeners have been notified. > - Assert.assertTrue("Should have been notified by legacy event dispatcher", this.hasBeenNotified); > - } > - > - @Test > - public void testLegacyDocumentSaveEventGetsDispatched() throws Exception > - { > - this.registerListenerWithLegacyEvent(new DocumentSaveEvent()); > - om.notify(new DocumentCreatedEvent(), new Object()); > - // The notification is synchronous, so the following assertion will only be tested > - // once all matching event listeners have been notified. > - Assert.assertTrue("Should have been notified by legacy event dispatcher", this.hasBeenNotified); > - } > - > - @Test > - public void testLegacyDocumentUpdateEventGetsDispatched() throws Exception > - { > - this.registerListenerWithLegacyEvent(new DocumentUpdateEvent()); > - om.notify(new DocumentUpdatedEvent(), new Object()); > - // The notification is synchronous, so the following assertion will only be tested > - // once all matching event listeners have been notified. > - Assert.assertTrue("Should have been notified by legacy event dispatcher", this.hasBeenNotified); > - } > - > - private void registerListenerWithLegacyEvent(final Event event) > - { > - this.om.addListener(new EventListener() > - { > - public String getName() > - { > - return "testLegacyEventDispatchEventListener"; > - } > - > - public List<Event> getEvents() > - { > - return Collections.<Event> singletonList(event); > - } > - > - public void onEvent(Event event, Object source, Object data) > - { > - setNotified(); > - } > - > - }); > - } > - > - private void setNotified() > - { > - this.hasBeenNotified = true; > - } > -} > > Copied: platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java (from rev 33375, platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java) > =================================================================== > --- platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java (rev 0) > +++ platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -0,0 +1,111 @@ > +/* > + * 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.observation.event; > + > +import java.util.Collections; > +import java.util.List; > + > +import junit.framework.Assert; > + > +import org.junit.Before; > +import org.junit.Test; > +import org.xwiki.bridge.event.DocumentCreatedEvent; > +import org.xwiki.bridge.event.DocumentDeletedEvent; > +import org.xwiki.bridge.event.DocumentUpdatedEvent; > +import org.xwiki.model.reference.DocumentReference; > +import org.xwiki.observation.EventListener; > +import org.xwiki.observation.ObservationManager; > +import org.xwiki.observation.event.DocumentDeleteEvent; > +import org.xwiki.observation.event.DocumentSaveEvent; > +import org.xwiki.observation.event.DocumentUpdateEvent; > +import org.xwiki.observation.event.Event; > +import org.xwiki.observation.event.filter.EventFilter; > +import org.xwiki.observation.event.filter.FixedNameEventFilter; > +import org.xwiki.test.AbstractComponentTestCase; > + > +public class LegacyEventDispatcherTest extends AbstractComponentTestCase > +{ > + private Event receivedEvent; > + > + private ObservationManager om; > + > + @Before > + public void setUp() throws Exception > + { > + this.om = getComponentManager().lookup(ObservationManager.class); > + } > + > + @Test > + public void testLegacyDocumentDeleteEventGetsDispatched() throws Exception > + { > + this.registerListenerWithLegacyEvent(new DocumentDeleteEvent()); > + this.om.notify(new DocumentDeletedEvent(new DocumentReference("wiki", "space", "name")), new Object()); > + > + // The notification is synchronous, so the following assertion will only be tested > + // once all matching event listeners have been notified. > + Assert.assertNotNull("Should have been notified by legacy event dispatcher", this.receivedEvent); > + Assert.assertEquals("Wrong event filter", "wiki:space.name", ((EventFilter)((FilterableEvent)this.receivedEvent).getEventFilter()).getFilter()); > + } > + > + @Test > + public void testLegacyDocumentSaveEventGetsDispatched() throws Exception > + { > + this.registerListenerWithLegacyEvent(new DocumentSaveEvent()); > + this.om.notify(new DocumentCreatedEvent(new DocumentReference("wiki", "space", "name")), new Object()); > + > + // The notification is synchronous, so the following assertion will only be tested > + // once all matching event listeners have been notified. > + Assert.assertNotNull("Should have been notified by legacy event dispatcher", this.receivedEvent); > + Assert.assertEquals("Wrong event filter", "wiki:space.name", ((EventFilter)((FilterableEvent)this.receivedEvent).getEventFilter()).getFilter()); > + } > + > + @Test > + public void testLegacyDocumentUpdateEventGetsDispatched() throws Exception > + { > + this.registerListenerWithLegacyEvent(new DocumentUpdateEvent()); > + this.om.notify(new DocumentUpdatedEvent(new DocumentReference("wiki", "space", "name")), new Object()); > + > + // The notification is synchronous, so the following assertion will only be tested > + // once all matching event listeners have been notified. > + Assert.assertNotNull("Should have been notified by legacy event dispatcher", this.receivedEvent); > + Assert.assertEquals("Wrong event filter", "wiki:space.name", ((EventFilter)((FilterableEvent)this.receivedEvent).getEventFilter()).getFilter()); > + } > + > + private void registerListenerWithLegacyEvent(final Event event) > + { > + this.om.addListener(new EventListener() > + { > + public String getName() > + { > + return "testLegacyEventDispatchEventListener"; > + } > + > + public List<Event> getEvents() > + { > + return Collections.<Event> singletonList(event); > + } > + > + public void onEvent(Event event, Object source, Object data) > + { > + receivedEvent = event; > + } > + }); > + } > +} > > > Property changes on: platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/LegacyEventDispatcherTest.java > ___________________________________________________________________ > Added: svn:keywords > + Id > > Deleted: platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java > =================================================================== > --- platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java 2010-12-14 09:19:33 UTC (rev 33375) > +++ platform/core/trunk/xwiki-model/src/main/java/org/xwiki/model/event/AbstractDocumentEvent.java 2010-12-14 14:01:28 UTC (rev 33376) > @@ -1,68 +0,0 @@ > -/* > - * 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.model.event; > - > -import org.xwiki.model.internal.reference.DefaultStringEntityReferenceSerializer; > -import org.xwiki.model.reference.DocumentReference; > -import org.xwiki.observation.event.AbstractFilterableEvent; > -import org.xwiki.observation.event.Event; > -import org.xwiki.observation.event.filter.EventFilter; > - > -/** > - * Base class for all document {@link Event events}. > - * > - * @version $Id$ > - */ > -public abstract class AbstractDocumentEvent extends AbstractFilterableEvent > -{ > - /** > - * The version identifier for this Serializable class. Increment only if the serialized form of the class > - * changes. > - */ > - private static final long serialVersionUID = 1L; > - > - /** > - * This event will match any other document event of the same type. > - */ > - public AbstractDocumentEvent() > - { > - super(); > - } > - > - /** > - * This event will match only events of the same type affecting the same document. > - * > - * @param documentReference the reference of the document relater to this event > - */ > - public AbstractDocumentEvent(DocumentReference documentReference) > - { > - super(new DefaultStringEntityReferenceSerializer().serialize(documentReference)); > - } > - > - /** > - * Constructor using a custom {@link EventFilter}. > - * > - * @param eventFilter the filter to use for matching events > - */ > - public AbstractDocumentEvent(EventFilter eventFilter) > - { > - super(eventFilter); > - } > -} > \ No newline at end of file > > _______________________________________________ > notifications mailing list > notifications(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/notifications >

14 years, 9 months

1
0
0 0

Re: [xwiki-devs] [xwiki-notifications] r33363 - in platform/core/trunk: xwiki-core/src/main/java/com/xpn/xwiki xwiki-core/src/main/java/com/xpn/xwiki/util xwiki-core/src/main/java/com/xpn/xwiki/web xwiki-velocity/src/main/java/org/xwiki/velocity/tools xwiki-xml xwiki-xml/src/main/java/org/xwiki/xml/internal xwiki-xml/src/main/resources/META-INF xwiki-xml/src/test/java/org/xwiki/xml/internal

by Vincent Massol

On Dec 13, 2010, at 4:36 PM, sdumitriu (SVN) wrote: > Author: sdumitriu > Date: 2010-12-13 16:36:16 +0100 (Mon, 13 Dec 2010) > New Revision: 33363 > > Added: > platform/core/trunk/xwiki-xml/src/main/java/org/xwiki/xml/internal/XMLScriptService.java > platform/core/trunk/xwiki-xml/src/test/java/org/xwiki/xml/internal/XMLScriptServiceTest.java > Modified: > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/XWiki.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/util/Util.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/web/Utils.java > platform/core/trunk/xwiki-velocity/src/main/java/org/xwiki/velocity/tools/EscapeTool.java > platform/core/trunk/xwiki-xml/pom.xml > platform/core/trunk/xwiki-xml/src/main/resources/META-INF/components.txt > Log: > XWIKI-5790: New script service for XML manipulation > Done. > > XWIKI-5514: "apostrophe" character badly displayed in Internet Explorer > XWIKI-5763: Remove entity encoding from UTF-8 text in XHTML > Better fix. [snip] > + /** > + * Construct a new (empty) DOM Document and return it. > + * > + * @return an empty DOM Document > + */ > + public Document getDomDocument() > + { > + try { > + return DocumentBuilderFactory.newInstance().newDocumentBuilder().newDocument(); > + } catch (ParserConfigurationException ex) { > + LOG.error("Cannot create DOM Documents", ex); > + return null; > + } > + } I don't like too much getXXX() naming for create methods. I prefer createXXX(). Thanks -Vincent [snip]

14 years, 9 months

2
1
0 0

Re: [xwiki-devs] [xwiki-notifications] r33366 - in platform/core/trunk/xwiki-component/xwiki-component-default/src: main/java/org/xwiki/component/embed test/java/org/xwiki/component/embed

by Vincent Massol

On Dec 13, 2010, at 5:16 PM, tmortagne (SVN) wrote: > Author: tmortagne > Date: 2010-12-13 17:16:56 +0100 (Mon, 13 Dec 2010) > New Revision: 33366 > > Modified: > platform/core/trunk/xwiki-component/xwiki-component-default/src/main/java/org/xwiki/component/embed/EmbeddableComponentManager.java > platform/core/trunk/xwiki-component/xwiki-component-default/src/test/java/org/xwiki/component/embed/EmbeddableComponentManagerTest.java > Log: > XWIKI-5792: ComponentManager#hasComponent does not work if the component has not already been lookuped > > Modified: platform/core/trunk/xwiki-component/xwiki-component-default/src/main/java/org/xwiki/component/embed/EmbeddableComponentManager.java > =================================================================== > --- platform/core/trunk/xwiki-component/xwiki-component-default/src/main/java/org/xwiki/component/embed/EmbeddableComponentManager.java 2010-12-13 16:03:58 UTC (rev 33365) > +++ platform/core/trunk/xwiki-component/xwiki-component-default/src/main/java/org/xwiki/component/embed/EmbeddableComponentManager.java 2010-12-13 16:16:56 UTC (rev 33366) > @@ -93,7 +93,7 @@ > */ > public <T> boolean hasComponent(Class<T> role, String roleHint) > { > - return this.components.containsKey(new RoleHint<T>(role, roleHint)); > + return this.descriptors.containsKey(new RoleHint<T>(role, roleHint)); Isn't this what we used to have and you changed it because you wanted hasComponent to return true if the component had been instantiated vs the component had been registered, no? Thanks -Vincent > } > > /** > @@ -103,7 +103,7 @@ > */ > public <T> boolean hasComponent(Class<T> role) > { > - return this.components.containsKey(new RoleHint<T>(role)); > + return this.descriptors.containsKey(new RoleHint<T>(role)); > } > > /** > > Modified: platform/core/trunk/xwiki-component/xwiki-component-default/src/test/java/org/xwiki/component/embed/EmbeddableComponentManagerTest.java > =================================================================== > --- platform/core/trunk/xwiki-component/xwiki-component-default/src/test/java/org/xwiki/component/embed/EmbeddableComponentManagerTest.java 2010-12-13 16:03:58 UTC (rev 33365) > +++ platform/core/trunk/xwiki-component/xwiki-component-default/src/test/java/org/xwiki/component/embed/EmbeddableComponentManagerTest.java 2010-12-13 16:16:56 UTC (rev 33366) > @@ -180,4 +180,18 @@ > parent.registerComponent(cd); > return parent; > } > + > + @Test > + public void testHasComponent() throws Exception > + { > + EmbeddableComponentManager ecm = new EmbeddableComponentManager(); > + > + DefaultComponentDescriptor<Role> d1 = new DefaultComponentDescriptor<Role>(); > + d1.setRole(Role.class); > + d1.setRoleHint("default"); > + ecm.registerComponent(d1); > + > + Assert.assertTrue(ecm.hasComponent(Role.class)); > + Assert.assertTrue(ecm.hasComponent(Role.class, "default")); > + } > }

14 years, 9 months

2
2
0 0

Re: [xwiki-devs] [xwiki-notifications] r33362 - platform/core/trunk/xwiki-extension/xwiki-extension-api/src/main/java/org/xwiki/extension/repository/internal

by Sergiu Dumitriu

On 12/13/2010 02:59 PM, tmortagne (SVN) wrote: > Author: tmortagne > Date: 2010-12-13 14:59:20 +0100 (Mon, 13 Dec 2010) > New Revision: 33362 > > Modified: > platform/core/trunk/xwiki-extension/xwiki-extension-api/src/main/java/org/xwiki/extension/repository/internal/DefaultLocalExtensionRepository.java > Log: > XWIKI-5772: Extension Manager use invalid character in extension files names > > Modified: platform/core/trunk/xwiki-extension/xwiki-extension-api/src/main/java/org/xwiki/extension/repository/internal/DefaultLocalExtensionRepository.java > =================================================================== > --- platform/core/trunk/xwiki-extension/xwiki-extension-api/src/main/java/org/xwiki/extension/repository/internal/DefaultLocalExtensionRepository.java 2010-12-13 13:39:11 UTC (rev 33361) > +++ platform/core/trunk/xwiki-extension/xwiki-extension-api/src/main/java/org/xwiki/extension/repository/internal/DefaultLocalExtensionRepository.java 2010-12-13 13:59:20 UTC (rev 33362) > @@ -23,6 +23,8 @@ > import java.io.FileOutputStream; > import java.io.FilenameFilter; > import java.io.IOException; > +import java.io.UnsupportedEncodingException; > +import java.net.URLEncoder; > import java.util.ArrayList; > import java.util.Collections; > import java.util.HashSet; > @@ -289,14 +291,26 @@ > > private File getFile(String id, String version, String type) > { > - return new File(getRootFolder(), id + "-" + version + "." + type); > + return new File(getRootFolder(), getFileName(id, version, type)); > } > > private File getDescriptorFile(String id, String version) > { > - return new File(getRootFolder(), id + "-" + version + ".xed"); > + return new File(getRootFolder(), getFileName(id, version, "xed")); > } > > + private String getFileName(String id, String version, String extension) > + { > + String fileName = id + "-" + version + "." + extension; > + try { Tabs here, replace with spaces: > + return URLEncoder.encode(fileName, "UTF-8"); > + } catch (UnsupportedEncodingException e) { > + // Should never happen > + > + return fileName; > + } > + } > + > private void addElement(Document document, Element parentElement, String elementName, String elementValue) > { > Element element = document.createElement(elementName); > -- Sergiu Dumitriu http://purl.org/net/sergiu/

14 years, 9 months

2
1
0 0

[xwiki-devs] Theme for the XWiki Enterprise 3.x cycle

by Vincent Massol

Hi guys, Just wanted to let everyone know that at XWiki SAS (http://xwiki.com) we've had a big brainstorm about what everyone thinks that XWiki Enterprise 3.x should have as the main tagline/direction. It wasn't an easy exercise but in the end we rallied around the following theme: Main Theme: "Building Apps and Distributing them" This means for example that the Extension Manager in progress is a key element of XE 3.x. It also means making it easier to create applications in XE. Secondary theme: "Polishing" This means finishing what was started in 2.x by making it even more polished, stable, speed improvement, etc. These are the 2 general themes that XWiki committers who are belonging to the XWiki SAS company will work on. Note that these are just themes and should be viewed as general directions and helpers when someone wonders what to tackle next. Feedback welcome, as always. Thanks -Vincent, on behalf of XWiki SAS Links to understand the relationship of XWiki SAS and the open source project: * http://www.xwiki.com/xwiki/bin/view/About/Values * http://www.xwiki.com/xwiki/bin/view/About/Manifesto

14 years, 9 months

1
0
0 0

[xwiki-devs] issue when creating a new action

by Ibrahim hammani

Hi, I've create a new action that I called 'PDFForm' , when I try to display a document using this action everything goes right except that XWiki displays an error and warn in the console (see below). For the warn, it happens cuz I used this code : try { fopFactory.setFontBaseURL(((Container) Utils.getComponent(Container.class)).getApplicationContext() .getResource("/WEB-INF/fonts/").getPath()); } catch (Throwable ex) { log.warn("Starting with 1.5, XWiki uses the WEB-INF/fonts/ directory as the font directory, " + "and it should contain the FreeFont (http://savannah.gnu.org/projects/freefont/) fonts. " + "FOP cannot access this directory. If this is an upgrade from a previous version, " + "make sure you also copy the WEB-INF/fonts directory from the new distribution package."); } This code doesn't display warn with 'pdf' action but with my new action. Can you tell me what i've missed please. Thank you. ______________________________ - Error creating form bean of class com.nearbeePDFForm java.lang.NullPointerException at org.apache.struts.config.FormBeanConfig.createActionForm(FormBeanConfig.java :212) at org.apache.struts.util.RequestUtils.createActionForm(RequestUtils.java:292) at org.apache.struts.util.RequestUtils.createActionForm(RequestUtils.java:191) at org.apache.struts.action.RequestProcessor.processActionForm(RequestProcessor .java:331) at org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:202) at org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196) at org.apache.struts.action.ActionServlet.doGet(ActionServlet.java:414) at javax.servlet.http.HttpServlet.service(HttpServlet.java:707) at javax.servlet.http.HttpServlet.service(HttpServlet.java:820) at org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:511) at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler .java:1166) at com.xpn.xwiki.web.ActionFilter.doFilter(ActionFilter.java:129) at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler .java:1157) at com.xpn.xwiki.wysiwyg.server.filter.ConversionFilter.doFilter(ConversionFilt er.java:152) at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler .java:1157) at com.xpn.xwiki.plugin.webdav.XWikiDavFilter.doFilter(XWikiDavFilter.java:68) at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler .java:1157) at org.xwiki.container.servlet.filters.internal.SavedRequestRestorerFilter.doFi lter(SavedRequestRestorerFilter.java:218) at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler .java:1157) at org.xwiki.container.servlet.filters.internal.SetCharacterEncodingFilter.doFi lter(SetCharacterEncodingFilter.java:112) at org.mortbay.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler .java:1157) at org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:388) at org.mortbay.jetty.security.SecurityHandler.handle(SecurityHandler.java:216) at org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:182) at org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:765) at org.mortbay.jetty.webapp.WebAppContext.handle(WebAppContext.java:418) at org.mortbay.jetty.handler.ContextHandlerCollection.handle(ContextHandlerColl ection.java:230) at org.mortbay.jetty.handler.HandlerCollection.handle(HandlerCollection.java:11 4) at org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:152) at org.mortbay.jetty.Server.handle(Server.java:326) at org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:536) at org.mortbay.jetty.HttpConnection$RequestHandler.headerComplete(HttpConnectio n.java:915) at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:539) at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:212) at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:405) at org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:409) at org.mortbay.thread.QueuedThreadPool$PoolThread.run(QueuedThreadPool.java:582 ) 2010-12-13 10:35:28,153 [http://localhost:8080/xwiki/bin/PDFForm/Sandbox/TestPage1] WARN impl.PdfExportImpl - Starting with 1.5, XWiki uses the WEB-INF/fonts/ directory as the font directory, and it should contain the FreeFont (http://savannah.gnu.org/projects/freefont/) fonts. FOP cannot access this directory. If this is an upgrade from a previous version, make sure you also copy the WEB-INF/fonts directory from the new distribution package.

14 years, 9 months

2
1
0 0

Re: [xwiki-devs] [xwiki-notifications] r33349 - in platform/core/trunk: . xwiki-annotations/xwiki-annotation-maintainer/src/main/java/org/xwiki/annotation/maintainer/internal xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge xwiki-bridge/src/m

by Jerome Velociter

On Mon, Dec 13, 2010 at 9:17 AM, Vincent Massol <vincent(a)massol.net> wrote: > > On Dec 11, 2010, at 11:37 AM, jvelociter (SVN) wrote: > > > Author: jvelociter > > Date: 2010-12-11 11:37:21 +0100 (Sat, 11 Dec 2010) > > New Revision: 33349 > > > > Added: > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/ > > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java > > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java > > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java > > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java > > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java > > > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java > > platform/core/trunk/xwiki-legacy/ > > platform/core/trunk/xwiki-legacy/pom.xml > > platform/core/trunk/xwiki-legacy/src/ > > platform/core/trunk/xwiki-legacy/src/main/ > > platform/core/trunk/xwiki-legacy/src/main/java/ > > platform/core/trunk/xwiki-legacy/src/main/java/org/ > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/ > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/ > > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java > > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/ > > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > > > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > > platform/core/trunk/xwiki-legacy/src/main/resources/ > > platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/ > > > platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt > > platform/core/trunk/xwiki-legacy/src/test/ > > platform/core/trunk/xwiki-legacy/src/test/java/ > > platform/core/trunk/xwiki-legacy/src/test/java/org/ > > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/ > > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/ > > > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/DocumentEventTest.java > > > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java > > > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/ > > Removed: > > > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > > > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > > > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > > > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/test/java/org/xwiki/observation/DocumentEventTest.java > > Modified: > > > platform/core/trunk/xwiki-annotations/xwiki-annotation-maintainer/src/main/java/org/xwiki/annotation/maintainer/internal/DocumentContentAnnotationUpdateListener.java > > platform/core/trunk/xwiki-bridge/pom.xml > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/XWiki.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/observation/remote/converter/DocumentEventConverter.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/PatternPlugin.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/rightsmanager/RightsManagerListener.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/render/XWikiMacrosMappingRenderer.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiCacheStore.java > > > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/user/impl/xwiki/XWikiGroupServiceImpl.java > > > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/XWikiTest.java > > > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/WikiMacrosTest.java > > > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/cache/DefaultDocumentCacheTest.java > > > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/cache/rendering/DefaultRenderingCacheTest.java > > > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/observation/remote/converter/DocumentEventConverterTest.java > > platform/core/trunk/xwiki-observation/xwiki-observation-api/pom.xml > > > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java > > > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-wikibridge/src/main/java/org/xwiki/rendering/internal/macro/wikibridge/WikiMacroEventListener.java > > Log: > > XWIKI-5786 Deprecate Document(Save|Update|Delete)Event and add new > Document(Created|Updated|Deleted)Event in xwiki-bridge instead > > XWIKI-5787 Add new Document(Creating|Updating|Deleting)Event events being > notified before the actual actions are performed on the document > > XWIKI-5788 New xwiki-legacy module to host backward compatibility code > > > Jerome, my bad i haven't followed the discussion too closely but could you > explain why you have used the xwiki-bridge module? > > The goal is to remove that module, not to add to it I think :) > > I'd have put the event bridge in xwiki-legacy. Why not put it there? > Hi Vincent, The goal is to have xwiki-legacy as backward-compatibility only. It means none of our modules should depend on it. We've decided to put the new events in xwiki-bridge since they get XWikiDocument passed as source and XWikiContext as data. When we can have new events that do not depend on xwiki-core (at runtime that is), we will create them in xwiki-model and move the ones in bridge in xwiki-legacy. Jerome. > > Thanks > -Vincent > _______________________________________________ > devs mailing list > devs(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/devs >

14 years, 9 months

3
8
0 0

Re: [xwiki-devs] [xwiki-notifications] Build failed in Hudson: xwiki-platform-core #6666

by Jerome Velociter

Seems my last commit introduced a cyclic dependency by making the observation API module depend on xwiki-model. Need to investigate how to solve that. The solution is maybe to move the AbstractDocumentEvent (the one that needed the DocumentReference thus xwiki-model) to xwiki-model directly. WDYT? Jerome. On Sat, Dec 11, 2010 at 4:59 PM, Hudson <build.noreply(a)xwiki.org> wrote: > See <http://hudson.xwiki.org/job/xwiki-platform-core/6666/changes> > > Changes: > > [Marius Dumitru Florea] XWIKI-5784: TempResource action fails in multiwiki > mode > Patch from Ludovic Dubost applied without changes. > > [Jerome Velociter] XWIKI-5786 Deprecate Document(Save|Update|Delete)Event > and add new Document(Created|Updated|Deleted)Event in xwiki-bridge instead > XWIKI-5787 Add new Document(Creating|Updating|Deleting)Event events being > notified before the actual actions are performed on the document > XWIKI-5788 New xwiki-legacy module to host backward compatibility code > > [Vincent Massol] Revert code committed by error in rev 32011 (XWIKI-5603) > > ------------------------------------------ > Started by an SCM change > Started by an SCM change > Started by an SCM change > Building remotely on agent-2 > Updating http://svn.xwiki.org/svnroot/xwiki/platform/core/trunk > A xwiki-bridge/src/main/java/org/xwiki/bridge/event > AU > xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java > AU > xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java > AU > xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java > AU > xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java > AU > xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java > AU > xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java > U xwiki-bridge/pom.xml > U > xwiki-annotations/xwiki-annotation-maintainer/src/main/java/org/xwiki/annotation/maintainer/internal/DocumentContentAnnotationUpdateListener.java > A xwiki-legacy > A xwiki-legacy/src > A xwiki-legacy/src/test > A xwiki-legacy/src/test/java > A xwiki-legacy/src/test/java/org > A xwiki-legacy/src/test/java/org/xwiki > A xwiki-legacy/src/test/java/org/xwiki/observation > A > xwiki-legacy/src/test/java/org/xwiki/observation/DocumentEventTest.java > A xwiki-legacy/src/test/java/org/xwiki/observation/event > AU > xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java > A xwiki-legacy/src/main > A xwiki-legacy/src/main/java > A xwiki-legacy/src/main/java/org > A xwiki-legacy/src/main/java/org/xwiki > A xwiki-legacy/src/main/java/org/xwiki/observation > A xwiki-legacy/src/main/java/org/xwiki/observation/event > A > xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > A > xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > A > xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > A > xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java > A xwiki-legacy/src/main/resources > A xwiki-legacy/src/main/resources/META-INF > A xwiki-legacy/src/main/resources/META-INF/components.txt > A xwiki-legacy/pom.xml > U > xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-wikibridge/src/main/java/org/xwiki/rendering/internal/macro/wikibridge/WikiMacroEventListener.java > U > xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-box/src/test/java/org/xwiki/rendering/RenderingTests.java > U xwiki-core/src/test/java/com/xpn/xwiki/XWikiTest.java > U > xwiki-core/src/test/java/com/xpn/xwiki/internal/cache/DefaultDocumentCacheTest.java > U > xwiki-core/src/test/java/com/xpn/xwiki/internal/cache/rendering/DefaultRenderingCacheTest.java > U > xwiki-core/src/test/java/com/xpn/xwiki/internal/WikiMacrosTest.java > U > xwiki-core/src/test/java/com/xpn/xwiki/internal/observation/remote/converter/DocumentEventConverterTest.java > U > xwiki-core/src/main/java/com/xpn/xwiki/render/XWikiMacrosMappingRenderer.java > U xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiCacheStore.java > U > xwiki-core/src/main/java/com/xpn/xwiki/plugin/rightsmanager/RightsManagerListener.java > U xwiki-core/src/main/java/com/xpn/xwiki/plugin/PatternPlugin.java > U xwiki-core/src/main/java/com/xpn/xwiki/XWiki.java > U > xwiki-core/src/main/java/com/xpn/xwiki/internal/observation/remote/converter/DocumentEventConverter.java > U > xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java > U > xwiki-core/src/main/java/com/xpn/xwiki/web/TempResourceAction.java > U > xwiki-core/src/main/java/com/xpn/xwiki/user/impl/xwiki/XWikiGroupServiceImpl.java > D > xwiki-observation/xwiki-observation-api/src/test/java/org/xwiki/observation/DocumentEventTest.java > D > xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > D > xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > D > xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > U > xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java > U xwiki-observation/xwiki-observation-api/pom.xml > At revision 33351 > Parsing POMs > [INFO ] snapshot > org.xwiki.platform:xwiki-core-rendering-parent:2.5-SNAPSHOT: checking for > updates from xwiki-externals > [INFO ] snapshot > org.xwiki.platform:xwiki-core-rendering-parent:2.5-SNAPSHOT: checking for > updates from xwiki-snapshots > [WARN ] *** CHECKSUM FAILED - Checksum failed on download: local = > '98149989ac8abf2f6e9fd52097531d2b3b1176a3'; remote = > '438d9fb1e32bf9e8242d17a5423f2023f311738f' - RETRYING > [WARN ] *** CHECKSUM FAILED - Checksum failed on download: local = > '98149989ac8abf2f6e9fd52097531d2b3b1176a3'; remote = > '438d9fb1e32bf9e8242d17a5423f2023f311738f' - IGNORING > [INFO ] snapshot org.xwiki.platform:xwiki-core-parent:2.5-SNAPSHOT: > checking for updates from xwiki-externals > [INFO ] snapshot org.xwiki.platform:xwiki-core-parent:2.5-SNAPSHOT: > checking for updates from xwiki-snapshots > [WARN ] *** CHECKSUM FAILED - Checksum failed on download: local = > 'a973942b53b3000c3ecad5925737e8c934defb8a'; remote = > 'f55c89977fc9cb996640a9945debda27c54f51f2' - RETRYING > [WARN ] *** CHECKSUM FAILED - Checksum failed on download: local = > 'a973942b53b3000c3ecad5925737e8c934defb8a'; remote = > 'f55c89977fc9cb996640a9945debda27c54f51f2' - IGNORING > [INFO ] snapshot org.xwiki.platform:xwiki-core-parent:2.5-SNAPSHOT: > checking for updates from reflections-repo > [INFO ] snapshot org.xwiki.platform:xwiki:37-SNAPSHOT: checking for updates > from xwiki-externals > [INFO ] snapshot org.xwiki.platform:xwiki:37-SNAPSHOT: checking for updates > from xwiki-snapshots > [WARN ] *** CHECKSUM FAILED - Checksum failed on download: local = > '63972618b17ed2541ac00bbf719b6f9affb79dca'; remote = > '165d5e99af55d7ce9def3572216cce4c2cf3d20b' - RETRYING > [WARN ] *** CHECKSUM FAILED - Checksum failed on download: local = > '63972618b17ed2541ac00bbf719b6f9affb79dca'; remote = > '165d5e99af55d7ce9def3572216cce4c2cf3d20b' - IGNORING > [INFO ] snapshot org.xwiki.platform:xwiki:37-SNAPSHOT: checking for updates > from reflections-repo > [trunk] $ /home/hudsonagent/jdk1.5.0_15/bin/java -Xmx1024m -cp > /home/hudsonagent/hudson_root/maven-agent.jar:/home/hudsonagent/maven/boot/classworlds-1.1.jar > hudson.maven.agent.Main /home/hudsonagent/maven > /home/hudsonagent/bin/slave.jar > /home/hudsonagent/hudson_root/maven-interceptor.jar 57975 > /home/hudsonagent/hudson_root/maven2.1-interceptor.jar > <===[HUDSON REMOTING CAPACITY]===> channel started > Executing Maven: -B -f < > http://hudson.xwiki.org/job/xwiki-platform-core/ws/trunk/pom.xml> clean > deploy -Pci -Pintegration-tests > [INFO] Scanning for projects... > [INFO] > ------------------------------------------------------------------------ > [ERROR] BUILD FAILURE > [INFO] > ------------------------------------------------------------------------ > [INFO] The projects in the reactor contain a cyclic reference: Edge between > 'Vertex{label='org.xwiki.platform:xwiki-core-script'}' and > 'Vertex{label='org.xwiki.platform:xwiki-core-observation-api'}' introduces > to cycle in the graph org.xwiki.platform:xwiki-core-observation-api --> > org.xwiki.platform:xwiki-core-model --> org.xwiki.platform:xwiki-core-script > --> org.xwiki.platform:xwiki-core-observation-api > [INFO] > ------------------------------------------------------------------------ > [INFO] For more information, run Maven with the -e switch > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 5 seconds > [INFO] Finished at: Sat Dec 11 16:59:29 GMT+01:00 2010 > [INFO] Final Memory: 9M/55M > [INFO] > ------------------------------------------------------------------------ > channel stopped > > _______________________________________________ > notifications mailing list > notifications(a)xwiki.org > http://lists.xwiki.org/mailman/listinfo/notifications >

14 years, 9 months

3
8
0 0

Re: [xwiki-devs] [xwiki-notifications] r33355 - in platform/core/trunk: xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge xwiki-core/src/main/java/com/xpn/xwiki/doc

by Vincent Massol

Hi Jerome, On Dec 11, 2010, at 10:51 PM, jvelociter (SVN) wrote: > Author: jvelociter > Date: 2010-12-11 22:51:45 +0100 (Sat, 11 Dec 2010) > New Revision: 33355 > > Modified: > platform/core/trunk/xwiki-bridge/pom.xml > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/DocumentAccessBridge.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/doc/DefaultDocumentAccessBridge.java > Log: > XWIKI-5789 New setProperty document access bridge method that takes document references instead of Strings > > > Modified: platform/core/trunk/xwiki-bridge/pom.xml > =================================================================== > --- platform/core/trunk/xwiki-bridge/pom.xml 2010-12-11 17:10:59 UTC (rev 33354) > +++ platform/core/trunk/xwiki-bridge/pom.xml 2010-12-11 21:51:45 UTC (rev 33355) > @@ -64,6 +64,19 @@ > </dependency> > </dependencies> > </plugin> > + <plugin> > + <groupId>org.codehaus.mojo</groupId> > + <artifactId>clirr-maven-plugin</artifactId> > + <configuration> > + <skip>${xwiki.clirr.skip}</skip> > + <comparisonVersion>${clirr.previous.version}</comparisonVersion> > + <excludes> > +  > + <exclude>org/xwiki/bridge/DocumentAccessBridge</exclude> > + </excludes> > + </configuration> > + </plugin> This shouldn't be placed here. All clirr exclusions must go in core/pom.xml Thanks -Vincent

14 years, 9 months

2
1
0 0

Re: [xwiki-devs] [xwiki-notifications] r33349 - in platform/core/trunk: . xwiki-annotations/xwiki-annotation-maintainer/src/main/java/org/xwiki/annotation/maintainer/internal xwiki-bridge xwiki-bridge/src/main/java/org/xwiki/bridge xwiki-bridge/src/main/java/org/xwiki/bridge/event xwiki-core/src/main/java/com/xpn/xwiki xwiki-core/src/main/java/com/xpn/xwiki/internal/cache xwiki-core/src/main/java/com/xpn/xwiki/internal/observation/remote/converter xwiki-core/src/main/java/com/xpn/xwiki/plugin xwiki-core/src

by Vincent Massol

On Dec 11, 2010, at 11:37 AM, jvelociter (SVN) wrote: > Author: jvelociter > Date: 2010-12-11 11:37:21 +0100 (Sat, 11 Dec 2010) > New Revision: 33349 > > Added: > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/ > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatedEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentCreatingEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletedEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentDeletingEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatedEvent.java > platform/core/trunk/xwiki-bridge/src/main/java/org/xwiki/bridge/event/DocumentUpdatingEvent.java > platform/core/trunk/xwiki-legacy/ > platform/core/trunk/xwiki-legacy/pom.xml > platform/core/trunk/xwiki-legacy/src/ > platform/core/trunk/xwiki-legacy/src/main/ > platform/core/trunk/xwiki-legacy/src/main/java/ > platform/core/trunk/xwiki-legacy/src/main/java/org/ > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/ > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/ > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/LegacyEventDispatcher.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/ > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > platform/core/trunk/xwiki-legacy/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > platform/core/trunk/xwiki-legacy/src/main/resources/ > platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/ > platform/core/trunk/xwiki-legacy/src/main/resources/META-INF/components.txt > platform/core/trunk/xwiki-legacy/src/test/ > platform/core/trunk/xwiki-legacy/src/test/java/ > platform/core/trunk/xwiki-legacy/src/test/java/org/ > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/ > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/ > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/DocumentEventTest.java > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/LegacyEventDispatcherTest.java > platform/core/trunk/xwiki-legacy/src/test/java/org/xwiki/observation/event/ > Removed: > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentDeleteEvent.java > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentSaveEvent.java > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/DocumentUpdateEvent.java > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/test/java/org/xwiki/observation/DocumentEventTest.java > Modified: > platform/core/trunk/xwiki-annotations/xwiki-annotation-maintainer/src/main/java/org/xwiki/annotation/maintainer/internal/DocumentContentAnnotationUpdateListener.java > platform/core/trunk/xwiki-bridge/pom.xml > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/XWiki.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/cache/DefaultDocumentCache.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/internal/observation/remote/converter/DocumentEventConverter.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/PatternPlugin.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/plugin/rightsmanager/RightsManagerListener.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/render/XWikiMacrosMappingRenderer.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiCacheStore.java > platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/user/impl/xwiki/XWikiGroupServiceImpl.java > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/XWikiTest.java > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/WikiMacrosTest.java > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/cache/DefaultDocumentCacheTest.java > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/cache/rendering/DefaultRenderingCacheTest.java > platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/internal/observation/remote/converter/DocumentEventConverterTest.java > platform/core/trunk/xwiki-observation/xwiki-observation-api/pom.xml > platform/core/trunk/xwiki-observation/xwiki-observation-api/src/main/java/org/xwiki/observation/event/AbstractDocumentEvent.java > platform/core/trunk/xwiki-rendering/xwiki-rendering-macros/xwiki-rendering-macro-wikibridge/src/main/java/org/xwiki/rendering/internal/macro/wikibridge/WikiMacroEventListener.java > Log: > XWIKI-5786 Deprecate Document(Save|Update|Delete)Event and add new Document(Created|Updated|Deleted)Event in xwiki-bridge instead > XWIKI-5787 Add new Document(Creating|Updating|Deleting)Event events being notified before the actual actions are performed on the document > XWIKI-5788 New xwiki-legacy module to host backward compatibility code Jerome, my bad i haven't followed the discussion too closely but could you explain why you have used the xwiki-bridge module? The goal is to remove that module, not to add to it I think :) I'd have put the event bridge in xwiki-legacy. Why not put it there? Thanks -Vincent

14 years, 9 months

1
0
0 0

[xwiki-devs] Autosuggest widget javascript

by Menso Heus

Hello everyone, I've created a form which needs to use autosuggest for fields. The form structure is basically a list of 10 things each of which has a supplier and a product. The supplier and product field are text input, and need to use autosuggest for pagenames from a specific space. I've been able to make the code from the Autosuggest widget work, which can be found here: http://platform.xwiki.org/xwiki/bin/view/DevGuide/AutoSuggestWidget I'm trying to translate the javascript offered there in the second codeblock on the page to a more generic function so I can say if the input is X, call suggestSupplier and if it is y call suggestProduct. All the supplier fields end in _Supplier in their name and all the product fields end in _SoftProd So far, I've been unsuccessful and I don't quite understand the structure of this javascript offered. Has someone already done this before and can they give me a pointer? Much appreciated, Menso

14 years, 9 months

3
3
0 0

[xwiki-devs] [discussion] Replace the box macro with the container macro

by Anca Luca

Hi devs, There is obviously some overlapping between the purposes of the box macro (http://extensions.xwiki.org/xwiki/bin/view/Extension/Box+Macro) and the container macro (http://extensions.xwiki.org/xwiki/bin/view/Extension/Container+Macro), namely that the box macro is just a very specific type of container. How do you feel about moving all or some of the box macros functionalities to the container macro and either implement the box macro as a particular type of container, or deprecate it completely? The way I see it is that the container macro would have parameters to allow to: * fix its size * render a border around * use a background color * title (to be discussed, I am not 100% convinced that it's generic enough) * image (to be discussed, same) * layout it inside text (float it to the left or right) * other decorations we could find useful for the users (note that devs could still write css if they want more). I know that all these functions can be accomplished by CSS, but so is the columned layout, for example, and the idea is to provide to the users an easy way to do it (in this case, WYSIWYG). I'm +1, regardless of the solution (deprecate or not), but I am very convinced of the overlapping, they should not be 2 different implementations. Thanks, Anca

14 years, 9 months

6
8
0 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

xwiki-devs