Merge pull request #935 from crazyserver/MOBILE-1987

MOBILE-1987 completion: Solve completion updating

Author: jleyva

www/addons/mod/chat/templates/chat.html

@@ -5,7 +5,7 @@
         <button ng-if="loaded" class="button button-icon icon ion-person-stalker" ng-click="showChatUsers()"></button>
     </ion-nav-buttons>
 
-    <ion-content class="padding" delegate-handle="mmaChatScroll" mm-state-class>
+    <ion-content delegate-handle="mmaChatScroll" mm-state-class>
         <mm-loading hide-until="loaded">
             <section class="mma-chat-discussion-container" aria-live="polite">
                 <div class="list">

www/core/components/course/directives/module.js

@@ -28,13 +28,14 @@ angular.module('mm.core.course')
  *
  * @example
  *
- * <mm-course-module module="module"></mm-course-module>
+ * <mm-course-module module="module" completion-changed="completionChanged"></mm-course-module>
  */
 .directive('mmCourseModule', function() {
     return {
         restrict: 'E',
         scope: {
             module: '=',
+            completionChanged: '=?'
         },
         templateUrl: 'core/components/course/templates/module.html'
     };

www/core/components/course/templates/section.html

@@ -22,7 +22,7 @@ <h2><mm-format-text watch="true">{{ section.name }}</mm-format-text></h2>
                     <mm-format-text>{{ section.summary }}</mm-format-text>
                 </div>
 
-                <mm-course-module class="item item-complex" ng-repeat="module in section.modules" module="module"></mm-course-module>
+                <mm-course-module class="item item-complex" ng-repeat="module in section.modules" module="module" completion-changed="completionChanged"></mm-course-module>
 
                 <!-- When there is nothing in the section. -->
                 <div class="item item-text-wrap" ng-if="section.modules.length < 1 && sections.length == 1">

www/core/directives/completion.js

@@ -102,11 +102,7 @@ angular.module('mm.core')
                             scope.afterChange();
                         }
                     }).catch(function(error) {
-                        if (error) {
-                            $mmUtil.showErrorModal(error);
-                        } else {
-                            $mmUtil.showErrorModal('mm.core.errorchangecompletion', true);
-                        }
+                        $mmUtil.showErrorModalDefault(error, 'mm.core.errorchangecompletion', true);
                     }).finally(function() {
                         modal.dismiss();
                     });