X-Git-Url: http://source.bookstackapp.com/bookstack/blobdiff_plain/2fd421b115132cdea97ec5ccbbab412e02659ab7..refs/pull/261/head:/resources/assets/js/controllers.js diff --git a/resources/assets/js/controllers.js b/resources/assets/js/controllers.js index 4763f9867..aebde8da4 100644 --- a/resources/assets/js/controllers.js +++ b/resources/assets/js/controllers.js @@ -272,7 +272,6 @@ module.exports = function (ngApp, events) { $scope.draftsEnabled = $attrs.draftsEnabled === 'true'; $scope.isUpdateDraft = Number($attrs.pageUpdateDraft) === 1; $scope.isNewPageDraft = Number($attrs.pageNewDraft) === 1; - $scope.commentsLoaded = false; // Set initial header draft text if ($scope.isUpdateDraft || $scope.isNewPageDraft) { @@ -682,19 +681,21 @@ module.exports = function (ngApp, events) { }]); - // CommentCrudController - ngApp.controller('CommentReplyController', ['$scope', '$http', function ($scope, $http) { + // Controller used to reply to and add new comments + ngApp.controller('CommentReplyController', ['$scope', '$http', '$timeout', function ($scope, $http, $timeout) { const MarkdownIt = require("markdown-it"); const md = new MarkdownIt({html: true}); let vm = this; - $scope.errors = {}; + vm.saveComment = function () { let pageId = $scope.comment.pageId || $scope.pageId; let comment = $scope.comment.text; + if (!comment) { + return events.emit('warning', trans('errors.empty_comment')); + } let commentHTML = md.render($scope.comment.text); let serviceUrl = `/ajax/page/${pageId}/comment/`; let httpMethod = 'post'; - let errorOp = 'add'; let reqObj = { text: comment, html: commentHTML @@ -704,20 +705,17 @@ module.exports = function (ngApp, events) { // this will be set when editing the comment. serviceUrl = `/ajax/page/${pageId}/comment/${$scope.comment.id}`; httpMethod = 'put'; - errorOp = 'update'; } else if ($scope.isReply === true) { // if its reply, get the parent comment id reqObj.parent_id = $scope.parentId; } $http[httpMethod](window.baseUrl(serviceUrl), reqObj).then(resp => { - if (!resp.data || resp.data.status !== 'success') { - return events.emit('error', trans('error')); + if (!isCommentOpSuccess(resp)) { + return; } + // hide the comments first, and then retrigger the refresh if ($scope.isEdit) { - $scope.comment.html = resp.data.comment.html; - $scope.comment.text = resp.data.comment.text; - $scope.comment.updated = resp.data.comment.updated; - $scope.comment.updated_by = resp.data.comment.updated_by; + updateComment($scope.comment, resp.data); $scope.$emit('evt.comment-success', $scope.comment.id); } else { $scope.comment.text = ''; @@ -728,86 +726,180 @@ module.exports = function (ngApp, events) { } $scope.$emit('evt.comment-success', null, true); } + $scope.comment.is_hidden = true; + $timeout(function() { + $scope.comment.is_hidden = false; + }); + events.emit('success', trans(resp.data.message)); - }, checkError(errorOp)); + }, checkError); }; - function checkError(errorGroupName) { - $scope.errors[errorGroupName] = {}; - return function(response) { - if (typeof response.data !== 'undefined' && typeof response.data.error !== 'undefined') { - events.emit('error', response.data.error); - } - if (typeof response.data !== 'undefined' && typeof response.data.validation !== 'undefined') { - $scope.errors[errorGroupName] = response.data.validation; - console.log($scope.errors[errorGroupName]) - } + function checkError(response) { + let msg = null; + if (isCommentOpSuccess(response)) { + // all good + return; + } else if (response.data) { + msg = response.data.message; + } else { + msg = trans('errors.comment_add'); + } + if (msg) { + events.emit('success', msg); } } }]); + // Controller used to delete comments + ngApp.controller('CommentDeleteController', ['$scope', '$http', '$timeout', function ($scope, $http, $timeout) { + let vm = this; - // CommentListController - ngApp.controller('CommentListController', ['$scope', '$http', '$timeout', function ($scope, $http, $timeout) { + vm.delete = function(comment) { + $http.delete(window.baseUrl(`/ajax/comment/${comment.id}`)).then(resp => { + if (!isCommentOpSuccess(resp)) { + return; + } + updateComment(comment, resp.data, $timeout, true); + }, function (resp) { + if (isCommentOpSuccess(resp)) { + events.emit('success', trans('entities.comment_deleted')); + } else { + events.emit('error', trans('error.comment_delete')); + } + }); + }; + }]); + + // Controller used to fetch all comments for a page + ngApp.controller('CommentListController', ['$scope', '$http', '$timeout', '$location', function ($scope, $http, $timeout, $location) { let vm = this; $scope.errors = {}; // keep track of comment levels $scope.level = 1; - vm.totalCommentsStr = 'Loading...'; + vm.totalCommentsStr = trans('entities.comments_loading'); vm.permissions = {}; + vm.trans = window.trans; $scope.$on('evt.new-comment', function (event, comment) { // add the comment to the comment list. vm.comments.push(comment); + ++vm.totalComments; + setTotalCommentMsg(); event.stopPropagation(); event.preventDefault(); }); - vm.canEdit = function (comment) { - if (vm.permissions.comment_update_all) { + vm.canEditDelete = function (comment, prop) { + if (!comment.active) { + return false; + } + let propAll = prop + '_all'; + let propOwn = prop + '_own'; + + if (vm.permissions[propAll]) { return true; } - if (vm.permissions.comment_update_own && comment.created_by.id === vm.current_user_id) { + if (vm.permissions[propOwn] && comment.created_by.id === vm.current_user_id) { return true; } + return false; - } + }; vm.canComment = function () { return vm.permissions.comment_create; - } + }; + + // check if there are is any direct linking + let linkedCommentId = $location.search().cm; $timeout(function() { $http.get(window.baseUrl(`/ajax/page/${$scope.pageId}/comments/`)).then(resp => { - if (!resp.data || resp.data.success !== true) { - // TODO : Handle error + if (!isCommentOpSuccess(resp)) { + // just show that no comments are available. + vm.totalComments = 0; + setTotalCommentMsg(); return; } vm.comments = resp.data.comments; - vm.totalComments = resp.data.total; + vm.totalComments = +resp.data.total; vm.permissions = resp.data.permissions; vm.current_user_id = resp.data.user_id; - - // TODO : Fetch message from translate. - if (vm.totalComments === 0) { - vm.totalCommentsStr = 'No comments found.'; - } else if (vm.totalComments === 1) { - vm.totalCommentsStr = '1 Comments'; - } else { - vm.totalCommentsStr = vm.totalComments + ' Comments' + setTotalCommentMsg(); + if (!linkedCommentId) { + return; } - }, checkError('app')); + $timeout(function() { + // wait for the UI to render. + focusLinkedComment(linkedCommentId); + }); + }, checkError); }); - function checkError(errorGroupName) { - $scope.errors[errorGroupName] = {}; - return function(response) { - console.log(response); + function setTotalCommentMsg () { + if (vm.totalComments === 0) { + vm.totalCommentsStr = trans('entities.no_comments'); + } else if (vm.totalComments === 1) { + vm.totalCommentsStr = trans('entities.one_comment'); + } else { + vm.totalCommentsStr = trans('entities.x_comments', { + numComments: vm.totalComments + }); + } + } + + function focusLinkedComment(linkedCommentId) { + let comment = angular.element('#' + linkedCommentId); + if (comment.length === 0) { + return; + } + + window.setupPageShow.goToText(linkedCommentId); + } + + function checkError(response) { + let msg = null; + if (isCommentOpSuccess(response)) { + // all good + return; + } else if (response.data) { + msg = response.data.message; + } else { + msg = trans('errors.comment_list'); + } + if (msg) { + events.emit('success', msg); } } }]); + function updateComment(comment, resp, $timeout, isDelete) { + comment.text = resp.comment.text; + comment.updated = resp.comment.updated; + comment.updated_by = resp.comment.updated_by; + comment.active = resp.comment.active; + if (isDelete && !resp.comment.active) { + comment.html = trans('entities.comment_deleted'); + } else { + comment.html = resp.comment.html; + } + if (!$timeout) { + return; + } + comment.is_hidden = true; + $timeout(function() { + comment.is_hidden = false; + }); + } + + function isCommentOpSuccess(resp) { + if (resp && resp.data && resp.data.status === 'success') { + return true; + } + return false; + } };