rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stanton Sievers" <siever...@gmail.com>
Subject Re: Review Request 17358: Cloning a page for the current user prompts them to accept/decline the page
Date Sun, 26 Jan 2014 00:36:18 GMT

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

(Updated Jan. 26, 2014, 12:36 a.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
-----

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

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


Testing (updated)
-------

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message