Re: [xwiki-devs] [xwiki-notifications] r27005 - platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer
13 Feb
2010
13 Feb
'10
5:14 p.m.
Hi Asiri,
On Feb 13, 2010, at 5:03 PM, asiri (SVN) wrote:
Author: asiri
Date: 2010-02-13 17:03:01 +0100 (Sat, 13 Feb 2010)
New Revision: 27005
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
Log:
* Added missing @Requirement annotation which was causing total failure of wysiwyg office
imports.
* This typo does not seem to be present in any other branches.
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
===================================================================
---
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
15:40:57 UTC (rev 27004)
+++
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
16:03:01 UTC (rev 27005)
@@ -67,6 +67,7 @@
/**
* The component used to parse attachment references.
*/
+ @Requirement
private AttachmentReferenceResolver<String> attachmentReferenceResolver;
Are you sure about this?
In the 2.2 branch we have:
/**
* The component used to parse attachment references.
*/
@Requirement("current")
private AttachmentReferenceResolver attachmentReferenceResolver;
Thanks
-Vincent
13 Feb
13 Feb
5:22 p.m.
New subject: [xwiki-devs] [xwiki-notifications] r27005 - platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer
Vincent Massol wrote:
Hi Asiri,
On Feb 13, 2010, at 5:03 PM, asiri (SVN) wrote:
My mistake. I removed the annotation in revision 26759. I wanted to
remove only the hint (there's no current document for GWT-RPC requests)
and it seems I removed the entire annotation..
Asiri, thanks for catching this!
Marius
Author: asiri
Date: 2010-02-13 17:03:01 +0100 (Sat, 13 Feb 2010)
New Revision: 27005
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
Log:
* Added missing @Requirement annotation which was causing total failure of wysiwyg office
imports.
* This typo does not seem to be present in any other branches.
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
===================================================================
---
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
15:40:57 UTC (rev 27004)
+++
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
16:03:01 UTC (rev 27005)
@@ -67,6 +67,7 @@
/**
* The component used to parse attachment references.
*/
+ @Requirement
private AttachmentReferenceResolver<String> attachmentReferenceResolver;
Are you sure about this?
In the 2.2 branch we have:
/**
* The component used to parse attachment references.
*/
@Requirement("current")
private AttachmentReferenceResolver attachmentReferenceResolver;
Thanks
-Vincent
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
5:33 p.m.
New subject: [xwiki-devs] [xwiki-notifications] r27005 - platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer
On Feb 13, 2010, at 5:22 PM, Marius Dumitru Florea wrote:
Vincent Massol wrote:
In this case it's still wrong in the 2.2 branch right now....
-Vincent
Hi Asiri,
On Feb 13, 2010, at 5:03 PM, asiri (SVN) wrote:
My mistake. I removed the annotation in revision 26759. I wanted to
remove only the hint (there's no current document for GWT-RPC requests)
and it seems I removed the entire annotation..
Asiri, thanks for catching this! Author: asiri
Date: 2010-02-13 17:03:01 +0100 (Sat, 13 Feb 2010)
New Revision: 27005
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
Log:
* Added missing @Requirement annotation which was causing total failure of wysiwyg office
imports.
* This typo does not seem to be present in any other branches.
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
===================================================================
---
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
15:40:57 UTC (rev 27004)
+++
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
16:03:01 UTC (rev 27005)
@@ -67,6 +67,7 @@
/**
* The component used to parse attachment references.
*/
+ @Requirement
private AttachmentReferenceResolver<String> attachmentReferenceResolver;
Are you sure about this?
In the 2.2 branch we have:
/**
* The component used to parse attachment references.
*/
@Requirement("current")
private AttachmentReferenceResolver attachmentReferenceResolver;
Marius
5:50 p.m.
New subject: [xwiki-devs] [xwiki-notifications] r27005 - platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer
Vincent Massol wrote:
On Feb 13, 2010, at 5:22 PM, Marius Dumitru Florea
wrote:
Vincent Massol wrote:
Hi Asiri,
On Feb 13, 2010, at 5:03 PM, asiri (SVN) wrote:
My mistake. I
removed the annotation in revision 26759. I wanted to
remove only the hint (there's no current document for GWT-RPC requests)
and it seems I removed the entire annotation..
Asiri, thanks for catching this! Author: asiri
Date: 2010-02-13 17:03:01 +0100 (Sat, 13 Feb 2010)
New Revision: 27005
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
Log:
* Added missing @Requirement annotation which was causing total failure of wysiwyg office
imports.
* This typo does not seem to be present in any other branches.
Modified:
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java
===================================================================
---
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
15:40:57 UTC (rev 27004)
+++
platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer/XWikiImportService.java 2010-02-13
16:03:01 UTC (rev 27005)
@@ -67,6 +67,7 @@
/**
* The component used to parse attachment references.
*/
+ @Requirement
private AttachmentReferenceResolver<String> attachmentReferenceResolver;
Are you sure about this?
In the 2.2 branch we have:
/**
* The component used to parse attachment references.
*/
@Requirement("current")
private AttachmentReferenceResolver attachmentReferenceResolver; In this case it's still wrong in the 2.2 branch
right now....
Theoretically yes, but the editor passes all the reference components on
the request so there shouldn't be any null parts to be resolved. So any
resolver should work. I removed the "current" hint just because it is
misleading.
Thanks,
Marius
-Vincent
Marius
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs 
14 Feb
14 Feb
3:34 a.m.
New subject: [xwiki-devs] [xwiki-notifications] r27005 - platform/web/trunk/xwiki-gwt-wysiwyg-server/src/main/java/com/xpn/xwiki/wysiwyg/server/plugin/importer
Hi,
I'll remove it from 2.2 branch as well.
Thanks.
- Asiri
Are you sure about this?
In the 2.2 branch we have:
/**
* The component used to parse attachment references.
*/
@Requirement("current")
private AttachmentReferenceResolver attachmentReferenceResolver;
My mistake. I
removed the annotation in revision 26759. I wanted to
remove only the hint (there's no current document for GWT-RPC requests)
and it seems I removed the entire annotation..
Asiri, thanks for catching this! In this case it's still wrong in the 2.2
branch right now....
Theoretically yes, but the editor passes all the reference components on
the request so there shouldn't be any null parts to be resolved. So any
resolver should work. I removed the "current" hint just because it is
misleading.
5331
days inactive
5332
days old
4 comments
3 participants
participants (3)
-
Asiri Rathnayake -
Marius Dumitru Florea -
Vincent Massol