Re: Review Request 17358: Cloning a page for the current user prompts them to accept/decline the page

[Stanton Sievers <si...@gmail.com>]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17358/
-----------------------------------------------------------

(Updated Feb. 18, 2014, 7:37 p.m.)


Review request for rave.


Bugs: RAVE-1090
    https://issues.apache.org/jira/browse/RAVE-1090


Repository: rave


Description
-------

If a page is cloned for the current user (which can be done via the REST APIs but not the current Share Page UI) the user is prompted to Accept/Decline the page on first render. 

The fix is to check and see if the current authenticated user is the user for which the page is being cloned.  If they are the same, set the page user status to "owned" instead of "pending".


Diffs (updated)
-----

  https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java 1569490 

Diff: https://reviews.apache.org/r/17358/diff/


Testing
-------

Existing unit tests pass.

Verified that cloning a page for another user still results in the prompt upon rendering the page.  Verified via Postman that cloning the page for the authenticated user results in no prompt upon first render of the page.

It's worth noting that I was having problems cloning pages with more than one region in the default portal.  Things worked as expected with a single column page layout where there is only one region.  Is this known behavior?  I did not encounter it when working on RAVE-1086.  I encountered it even without this or any other patches.


Thanks,

Stanton Sievers