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

Review request for rave.


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


Repository: rave


Description
-------

When cloning a page, the PageApi and PageService will now return the Page 
object itself.


Diffs
-----

  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/PageService.java
 1561424 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java
 1561424 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/service/impl/DefaultPageServiceTest.java
 1561424 
  
https://svn.apache.org/repos/asf/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/api/rpc/PageApi.java
 1561424 

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


Testing
-------

Existing unit tests pass.  Added a new unit test to sanity check the returned 
value.

Verified that page cloning works in the default portal and that the REST call 
returns the correct page object.

There's definitely more unit testing that needs to be done around the page 
cloning API.


Thanks,

Stanton Sievers

Reply via email to