The Angular.js code in Hubbub.js functions $scope.updatePost and $scope.deletePost contains references to an unscoped isEditScope variable. The unscoped variable looks like a remnant of some earlier refactoring.
Since the isEditState variable is unscoped (rather than being referenced as $scope.isEditState), it won't bind to the Angular UI directive for ng-show in the gsp, and will have no visual effect.
There is already code in the update callback inside updatePost() that correctly sets the state of $scope.isEditState on update, so the unscoped isEditState = false line is entirely unnecessary (and misleading) in both updatePost() and deletePost().
The Angular.js code in Hubbub.js functions
$scope.updatePostand$scope.deletePostcontains references to an unscopedisEditScopevariable. The unscoped variable looks like a remnant of some earlier refactoring.Since the
isEditStatevariable is unscoped (rather than being referenced as$scope.isEditState), it won't bind to the Angular UI directive forng-showin the gsp, and will have no visual effect.There is already code in the update callback inside
updatePost()that correctly sets the state of$scope.isEditStateon update, so the unscopedisEditState = falseline is entirely unnecessary (and misleading) in bothupdatePost()anddeletePost().