On Feb 10, 2011, at 8:50 PM, jvelociter (SVN) wrote:

> Author: jvelociter
> Date: 2011-02-10 20:50:06 +0100 (Thu, 10 Feb 2011)
> New Revision: 34616
> 
> Added:
>  enterprise/trunk/wiki/src/main/resources/XWiki/RequiredRightClass.xml
> Modified:
>  enterprise/trunk/wiki/src/main/resources/Main/Activity.xml
>  enterprise/trunk/wiki/src/main/resources/Main/Spaces.xml
>  enterprise/trunk/wiki/src/main/resources/XWiki/AllAttachmentsResults.xml
>  enterprise/trunk/wiki/src/main/resources/XWiki/DeletedAttachments.xml
>  enterprise/trunk/wiki/src/main/resources/XWiki/DeletedDocuments.xml
> Log:
> XE-831 Provide a XClass that allow to mark pages that require to be saved by 
> a user with certain rights
> XE-832 Add XWiki.RequiredRightClass to XE documents that require to be saved 
> with certain rights
> 

[snip]

> Modified: enterprise/trunk/wiki/src/main/resources/Main/Spaces.xml
> ===================================================================
> --- enterprise/trunk/wiki/src/main/resources/Main/Spaces.xml  2011-02-10 
> 19:49:34 UTC (rev 34615)
> +++ enterprise/trunk/wiki/src/main/resources/Main/Spaces.xml  2011-02-10 
> 19:50:06 UTC (rev 34616)
> @@ -1,5 +1,4 @@
> <?xml version="1.0" encoding="UTF-8"?>

[snip]

> +<className>XWiki.RequiredRightClass</className>
> +<guid>eb832884-79a9-4e96-b1fa-55fdbb06b0a5</guid>
> +<property>
> +<level>edit</level>

I don't understand the point of settings RequiredRightClass with level = edit. 
Editing a page always requires edit rights anyway, no?

(I'm sure I'm missing something though...)

Note: Wiki/TagCloud.xml also has an "edit" level

> Added: enterprise/trunk/wiki/src/main/resources/XWiki/RequiredRightClass.xml
> ===================================================================
> --- enterprise/trunk/wiki/src/main/resources/XWiki/RequiredRightClass.xml     
>                         (rev 0)
> +++ enterprise/trunk/wiki/src/main/resources/XWiki/RequiredRightClass.xml     
> 2011-02-10 19:50:06 UTC (rev 34616)

Hmm do we consider the RequiredRightClass a "core" class or not? If so I think 
we should create it programmatically if it doesn't exist in the wiki at startup.

Thanks
-Vincent

_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to