Tobias Jeger pushed to branch feature/cmng-psp1-CHANNELMGR-688 at cms-community 
/ hippo-addon-channel-manager


Commits:
e27625a9 by Tobias Jeger at 2016-05-24T13:30:56+02:00
CHANNELMGR-688 Rework after review

- - - - -


5 changed files:

- frontend-ng/src/angularjs/channel/page/actions/pageActions.controller.js
- frontend-ng/src/angularjs/channel/page/actions/pageActions.spec.js
- frontend-ng/src/angularjs/utils/directive/illegalCharacters.directive.spec.js
- frontend-ng/src/i18n/en.json
- frontend-ng/src/i18n/nl.json


Changes:

=====================================
frontend-ng/src/angularjs/channel/page/actions/pageActions.controller.js
=====================================
--- a/frontend-ng/src/angularjs/channel/page/actions/pageActions.controller.js
+++ b/frontend-ng/src/angularjs/channel/page/actions/pageActions.controller.js
@@ -125,9 +125,9 @@ export class PageActionsCtrl {
       // do nothing on cancel
   }
 
-  _confirmDelete(pageTitle) {
+  _confirmDelete(page) {
     const confirm = this.DialogService.confirm()
-      .title(`${this.$translate.instant('DELETE')} '${pageTitle}'?`)
+      .title(this.$translate.instant('CONFIRM_DELETE_PAGE_MESSAGE', { page }))
       .ok(this.$translate.instant('DELETE'))
       .cancel(this.$translate.instant('CANCEL'));
 


=====================================
frontend-ng/src/angularjs/channel/page/actions/pageActions.spec.js
=====================================
--- a/frontend-ng/src/angularjs/channel/page/actions/pageActions.spec.js
+++ b/frontend-ng/src/angularjs/channel/page/actions/pageActions.spec.js
@@ -195,12 +195,6 @@ describe('PageActions', () => {
     expect(copyAction.isEnabled()).toBe(true);
   });
 
-  it('builds the action label correctly', () => {
-    SiteMapItemService.isEditable.and.returnValue(false);
-
-    SiteMapItemService.isEditable.and.returnValue(true);
-  });
-
   it('does nothing when not confirming the deletion of a page', () => {
     const PageActionsCtrl = compileDirectiveAndGetController();
     const deleteAction = PageActionsCtrl.actions.find((action) => action.id 
=== 'delete');


=====================================
frontend-ng/src/angularjs/utils/directive/illegalCharacters.directive.spec.js
=====================================
--- 
a/frontend-ng/src/angularjs/utils/directive/illegalCharacters.directive.spec.js
+++ 
b/frontend-ng/src/angularjs/utils/directive/illegalCharacters.directive.spec.js
@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-describe('PageActionCreate', () => {
+describe('IllegalCharactersDirective', () => {
   'use strict';
 
   let $rootScope;


=====================================
frontend-ng/src/i18n/en.json
=====================================
--- a/frontend-ng/src/i18n/en.json
+++ b/frontend-ng/src/i18n/en.json
@@ -6,6 +6,7 @@
   "CHANNEL_PROPERTIES_SAVE_SUCCESS": "Channel settings saved successfully",
   "CONFIRM_DELETE_COMPONENT_MESSAGE": "Are you sure you want to remove the 
component '{{component}}' from this template?",
   "CONFIRM_DELETE_COMPONENT_TITLE": "Confirm remove",
+  "CONFIRM_DELETE_PAGE_MESSAGE": "Delete '{{page}}'?",
   "CONFIRM_DISCARD_SELECTED_CHANGES_MESSAGE": "Discard all selected changes?",
   "CONFIRM_DISCARD_OWN_CHANGES_MESSAGE": "Are you sure you want to discard all 
your unpublished changes to this channel?",
   "CONFIRM_DISCARD_CHANGES_TITLE": "Confirm discard changes",


=====================================
frontend-ng/src/i18n/nl.json
=====================================
--- a/frontend-ng/src/i18n/nl.json
+++ b/frontend-ng/src/i18n/nl.json
@@ -6,6 +6,7 @@
   "CHANNEL_PROPERTIES_SAVE_SUCCESS": "???",
   "CONFIRM_DELETE_COMPONENT_MESSAGE": "Weet u zeker dat u {0} wilt 
verwijderen?",
   "CONFIRM_DELETE_COMPONENT_TITLE": "Bevestig het verwijderen",
+  "CONFIRM_DELETE_PAGE_MESSAGE": "'{{page}}' verwijderen?",
   "CONFIRM_DISCARD_SELECTED_CHANGES_MESSAGE": "Maak de geselecteerde 
wijzigingen ongedaan?",
   "CONFIRM_DISCARD_OWN_CHANGES_MESSAGE": "Weet u zeker dat u al uw 
ongepubliceerde veranderingen in dit channel ongedaan wilt maken?",
   "CONFIRM_DISCARD_CHANGES_TITLE": "Bevestig ongedaan maken",



View it on GitLab: 
https://code.onehippo.org/cms-community/hippo-addon-channel-manager/commit/e27625a92c18d060670ed46ed401d613e594150b
_______________________________________________
Hippocms-svn mailing list
Hippocms-svn@lists.onehippo.org
https://lists.onehippo.org/mailman/listinfo/hippocms-svn

Reply via email to