25 Sep
2018
25 Sep
'18
11:34 a.m.
On Sun, Sep 23, 2018 at 11:12 AM Vincent Massol <vincent(a)massol.net> wrote:
Hi Simon,
On 21 Sep 2018, at 16:58, Simon Urli
<simon.urli(a)xwiki.com> wrote:
On 9/21/18 4:53 PM, Adel Atallah wrote:
renaming/moving page
(using forged URL)
+1 for the warning, but I would not forbid simple
users from renaming
or moving pages but instead just hide the action (from the page menu).
OK I should have written it: by "forbid" I meant:
1. Hide the action from the menu
2. Return an error message if the user try to access the
So you suggest we shouldn't do 2?
So +1 to prevent/warn the user when doing a move/renaming AND copy pages containing XClass definitions
FTR, copying a single page having an XClass definition is not dangerous (it
won't break the application that owns the page), as it only creates a new
class definition. Copying an entire application is not dangerous either.
The copy won't work like the original application (this justifies a warning
as it may fail the user expectations), but the original application will
still work. Renaming or moving an application is dangerous as it breaks the
application.
(the message should list all such pages).
-1 to hide the action from the menu (if you’re talking about the
“Move/Rename” and “Copy" actions) because:
1) you get to choose whether you move/rename/copy children after you click
the action
2) even when the current page has an XClass, the user wouldn't understand
why he cannot see/click on the action. It’s better that he can do it but
get an error message, explaining why and telling him that to contact an
advanced users if he really needs to do it.
Thanks
-Vincent
> On Fri, Sep 21, 2018 at 4:44 PM Simon Urli <simon.urli(a)xwiki.com>
wrote:
>>
>> Hi all,
>>
>> users might currently break their AWM application by renaming/moving
>> pages containing XClass definition.
>>
>> We need a proper refactoring operation to be able to properly do such
>> move/rename. But this feature might take a while to be completely
>> available.
>>
>> In the meantime I propose that we prevent users from renaming/moving
>> pages containing XClass.
>>
>> What I propose is the following:
>> - Forbid completely *simple users* to rename/move pages containing
XClass
>> - Display a warning to *advanced users*
when they perform such
>> operation: the same kind of warning we already have when performing
edit
> on
XWiki pages
>
> WDYT?
>
> Simon
>
> --
> Simon Urli
> Software Engineer at XWiki SAS
> simon.urli(a)xwiki.com
> More about us at http://www.xwiki.com
--
Simon Urli
Software Engineer at XWiki SAS
simon.urli(a)xwiki.com
More about us at http://www.xwiki.com