Bladeren bron

Make label removal also work in compare mode.

Paulo Reis 8 jaren geleden
bovenliggende
commit
b0261970fc

+ 23 - 0
services/web/app/views/project/editor/history.pug

@@ -93,3 +93,26 @@ script(type="text/ng-template", id="historyLabelTooltipTpl")
 			|  {{ $ctrl.labelText }}
 			|  {{ $ctrl.labelText }}
 		p.history-label-tooltip-owner #{translate("history_label_created_by")} {{ $ctrl.labelOwnerName }}
 		p.history-label-tooltip-owner #{translate("history_label_created_by")} {{ $ctrl.labelOwnerName }}
 		time.history-label-tooltip-datetime {{ labelCreationDateTime | formatDate }}
 		time.history-label-tooltip-datetime {{ labelCreationDateTime | formatDate }}
+
+
+script(type="text/ng-template", id="historyV2DeleteLabelModalTemplate")
+	.modal-header
+		h3 #{translate("history_delete_label")}
+	.modal-body
+		.alert.alert-danger(ng-show="state.error.message") {{ state.error.message}} 
+		.alert.alert-danger(ng-show="state.error && !state.error.message") #{translate("generic_something_went_wrong")}
+		p.help-block(ng-if="labelDetails")
+			| #{translate("history_are_you_sure_delete_label")}
+			strong "{{ labelDetails.comment }}"
+			| ?
+	.modal-footer
+		button.btn.btn-default(
+			type="button"
+			ng-disabled="state.inflight"
+			ng-click="$dismiss()"
+		) #{translate("cancel")}
+		button.btn.btn-primary(
+			type="button"
+			ng-click="deleteLabel()"
+			ng-disabled="state.inflight"
+		) {{ state.inflight ? '#{translate("history_deleting_label")}' : '#{translate("history_delete_label")}' }}

+ 1 - 1
services/web/app/views/project/editor/history/entriesListV2.pug

@@ -72,7 +72,7 @@ aside.change-list(
 						label-text="label.comment"
 						label-text="label.comment"
 						label-owner-name="getDisplayNameById(label.user_id)"
 						label-owner-name="getDisplayNameById(label.user_id)"
 						label-creation-date-time="label.created_at"
 						label-creation-date-time="label.created_at"
-						is-owned-by-current-user="label.user_id === $ctrl.currentUser.id"
+						is-owned-by-current-user="label.user_id === user.id"
 						on-label-delete="deleteLabel(label)"
 						on-label-delete="deleteLabel(label)"
 					)
 					)
 					div.time {{ update.meta.end_ts | formatDate:'h:mm a' }}
 					div.time {{ update.meta.end_ts | formatDate:'h:mm a' }}

+ 1 - 23
services/web/app/views/project/editor/history/toolbarV2.pug

@@ -52,26 +52,4 @@ script(type="text/ng-template", id="historyV2AddLabelModalTemplate")
 				ng-disabled="addLabelModalForm.$invalid || state.inflight"
 				ng-disabled="addLabelModalForm.$invalid || state.inflight"
 				ng-value="state.inflight ? '" + translate("history_adding_label") + "' : '" + translate("history_add_label") + "'"
 				ng-value="state.inflight ? '" + translate("history_adding_label") + "' : '" + translate("history_add_label") + "'"
 				type="submit"
 				type="submit"
-			)
-
-script(type="text/ng-template", id="historyV2DeleteLabelModalTemplate")
-	.modal-header
-		h3 #{translate("history_delete_label")}
-	.modal-body
-		.alert.alert-danger(ng-show="state.error.message") {{ state.error.message}} 
-		.alert.alert-danger(ng-show="state.error && !state.error.message") #{translate("generic_something_went_wrong")}
-		p.help-block(ng-if="labelDetails")
-			| #{translate("history_are_you_sure_delete_label")}: 
-			strong "{{ labelDetails.comment }}"
-			| ?
-	.modal-footer
-		button.btn.btn-default(
-			type="button"
-			ng-disabled="state.inflight"
-			ng-click="$dismiss()"
-		) #{translate("cancel")}
-		button.btn.btn-primary(
-			type="button"
-			ng-click="deleteLabel()"
-			ng-disabled="state.inflight"
-		) {{ state.inflight ? '#{translate("history_deleting_label")}' : '#{translate("history_delete_label")}' }}
+			)

+ 3 - 0
services/web/public/coffee/ide/history/components/historyEntry.coffee

@@ -4,6 +4,9 @@ define [
 ], (App, displayNameForUser) ->
 ], (App, displayNameForUser) ->
 	historyEntryController = ($scope, $element, $attrs, _) ->
 	historyEntryController = ($scope, $element, $attrs, _) ->
 		ctrl = @
 		ctrl = @
+		# This method (and maybe the one below) will be removed soon. User details data will be 
+		# injected into the history API responses, so we won't need to fetch user data from other
+		# local data structures.
 		_getUserById = (id) ->
 		_getUserById = (id) ->
 			_.find ctrl.users, (user) ->
 			_.find ctrl.users, (user) ->
 				curUserId = user?._id or user?.id
 				curUserId = user?._id or user?.id

+ 3 - 0
services/web/public/coffee/ide/history/controllers/HistoryListController.coffee

@@ -8,6 +8,9 @@ define [
 
 
 		projectUsers = $scope.project.members.concat $scope.project.owner
 		projectUsers = $scope.project.members.concat $scope.project.owner
 
 
+		# This method (and maybe the one below) will be removed soon. User details data will be 
+		# injected into the history API responses, so we won't need to fetch user data from other
+		# local data structures.
 		_getUserById = (id) ->
 		_getUserById = (id) ->
 			_.find projectUsers, (user) ->
 			_.find projectUsers, (user) ->
 				curUserId = user?._id or user?.id
 				curUserId = user?._id or user?.id