----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4843/#review7345 -----------------------------------------------------------
/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/Page.java <https://reviews.apache.org/r/4843/#comment16223> If this query is no longer necessary, please remove it /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/repository/PageUserRepository.java <https://reviews.apache.org/r/4843/#comment16224> Please add javadocs for all methods /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java <https://reviews.apache.org/r/4843/#comment16225> Is this still a TODO? When I delete a shared page, it's also deleted for the users I've shared it with. /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java <https://reviews.apache.org/r/4843/#comment16226> Minor improvement: use Boolean.TRUE and Boolean.FALSE /trunk/rave-portal-resources/src/main/resources/messages.properties <https://reviews.apache.org/r/4843/#comment16227> Dutch translations: page.general.sharepage=Deel pagina page.general.removeshare=Trek delen in page.general.search.title=Zoek of blader voor gebruiker /trunk/rave-portal-resources/src/main/resources/messages.properties <https://reviews.apache.org/r/4843/#comment16228> Dutch translations: sharing.dialog.confirm.title=Nieuwe gedeelde pagina gevonden sharing.dialog.confirm.message={0} heeft een pagina met jou gedeeld. Wil je deze gedeelde pagina toevoegen? 337 sharing.page.tab.icon.tip=Deze pagina is met jou gedeeld door {0}. /trunk/rave-portal-resources/src/main/resources/messages.properties <https://reviews.apache.org/r/4843/#comment16229> Dutch: Zoekterm mag niet leeg zijn /trunk/rave-portal-resources/src/main/resources/messages.properties <https://reviews.apache.org/r/4843/#comment16230> Dutch: Verwijder /trunk/rave-portal-resources/src/main/resources/messages.properties <https://reviews.apache.org/r/4843/#comment16231> Dutch: _rave_client.common.accept=Accepteer _rave_client.common.decline=Wijs af _rave_client.common.username=Gebruikersnaam _rave_client.common.sharing=Delen /trunk/rave-portal-resources/src/main/resources/messages.properties <https://reviews.apache.org/r/4843/#comment16232> Dutch: _rave_client.search.list.result.x.to.y=Toont resultaat {0} - {1} van {2} _rave_client.create.share=Gebruiker is toegevoegd aan deze pagina _rave_client.revoke.share=Gebruiker is verwijderd van deze pagina _rave_client.create.share.confirm=Weet je zeker dat je deze gebruiker wilt toevoegen? _rave_client.revoke.share.confirm=Weet je zeker dat je deze gebruiker wilt verwijderen? 374 _rave_client.revoke.share.current.user=Deze pagina is niet langer gedeeld. _rave_client.revoke.share.current.user.confirm=Weet je zeker dat je deze gedeelde pagina wilt verwijderen? _rave_client.no.results.found=Geen resultaten gevonden - Jasha On 2012-04-27 09:42:17, Paul Sharples wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4843/ > ----------------------------------------------------------- > > (Updated 2012-04-27 09:42:17) > > > Review request for rave. > > > Summary > ------- > > RAVE-103. support shared spaces. I've submitted this patch here rather than > commit the code directly, as the changes affect the UI and I wanted a request > for comments type approach first. This is a page sharing patch which allows a > user to share his/her page with other rave users, as well as also allowing > the user to revoke page shares. A user who receives a shared page can opt to > confirm the share (meaning the page will always appear in his/her tabbed page > list or decline it (i.e I don't want this shared page). There's still more > to improve on this, but the basic functionality is there. Note I have removed > the render sequencing away from the page object into the new pageUser object. > This is because with the possibility of having several users of a page, they > all need to have their own page render sequencing. See RAVE-103. > > > Diffs > ----- > > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/Page.java > 1306906 > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/PageStatus.java > PRE-CREATION > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/PageUser.java > PRE-CREATION > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/repository/PageRepository.java > 1308947 > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/repository/PageUserRepository.java > PRE-CREATION > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/repository/impl/JpaPageRepository.java > 1308947 > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/repository/impl/JpaPageUserRepository.java > PRE-CREATION > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/security/impl/DefaultPagePermissionEvaluator.java > 1306906 > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/PageService.java > 1306906 > > /trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPageService.java > 1310534 > > /trunk/rave-components/rave-core/src/main/resources/META-INF/persistence.xml > 1306906 > > /trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/model/PageTest.java > 1306906 > > /trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/repository/impl/JpaPageRepositoryTest.java > 1308947 > > /trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/service/impl/DefaultPageServiceTest.java > 1327947 > /trunk/rave-components/rave-core/src/test/resources/test_data.sql 1308947 > > /trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/api/rpc/PageApi.java > 1306906 > > /trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/api/rpc/UserApi.java > PRE-CREATION > > /trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/PageController.java > 1306906 > > /trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/util/ModelKeys.java > 1330718 > > /trunk/rave-components/rave-web/src/test/java/org/apache/rave/portal/web/controller/PageControllerTest.java > 1330044 > /trunk/rave-portal-resources/src/main/resources/messages.properties 1330724 > /trunk/rave-portal-resources/src/main/webapp/WEB-INF/db/initial_data.sql > 1327941 > /trunk/rave-portal-resources/src/main/webapp/WEB-INF/jsp/views/page.jsp > 1330737 > /trunk/rave-portal-resources/src/main/webapp/css/default.css 1330727 > /trunk/rave-portal-resources/src/main/webapp/script/rave_api.js 1306906 > /trunk/rave-portal-resources/src/main/webapp/script/rave_layout.js 1330737 > /trunk/rave-portal/src/test/resources/test-data.sql 1306906 > > Diff: https://reviews.apache.org/r/4843/diff > > > Testing > ------- > > > Thanks, > > Paul > >
