From 5bc7fcd8d00758b4cb3499ab8c077e7e2ad21d72 Mon Sep 17 00:00:00 2001
From: Gustavo Maximiliano Cortez <cmgustavo83@gmail.com>
Date: Fri, 16 Jan 2015 03:41:54 -0300
Subject: [PATCH] Fix PIN create. Fix spinner when creating profile

---
 js/controllers/createProfile.js | 14 ++++-------
 js/services/identityService.js  |  4 +++-
 views/createProfile.html        | 41 ++++++++++++++++-----------------
 3 files changed, 27 insertions(+), 32 deletions(-)

diff --git a/js/controllers/createProfile.js b/js/controllers/createProfile.js
index cb16a0559..a34ad8796 100644
--- a/js/controllers/createProfile.js
+++ b/js/controllers/createProfile.js
@@ -41,13 +41,6 @@ angular.module('copayApp.controllers').controller('CreateProfileController', fun
         $scope.passwordStrength = null;
 
         _firstpin = null;
-
-        $scope.setPinForm.newpin.$setViewValue('');
-        $scope.setPinForm.newpin.$render();
-        $scope.setPinForm.repeatpin.$setViewValue('');
-        $scope.setPinForm.repeatpin.$render();
-        $scope.setPinForm.$setPristine();
-
         $scope.error = 'Entered PINs were not equal. Try again';
         $timeout(function() {
           $scope.$digest();
@@ -79,7 +72,6 @@ angular.module('copayApp.controllers').controller('CreateProfileController', fun
         _credentials = null;
         $scope.askForPin = 0;
         $rootScope.hasPin = true;
-        $rootScope.starting = null;
         $scope.createDefaultWallet();
       });
     }, 100);
@@ -123,8 +115,10 @@ angular.module('copayApp.controllers').controller('CreateProfileController', fun
   /* Last step. Will emit after creation so the UX gets updated */
   $scope.createDefaultWallet = function() {
     $rootScope.hideNavigation = false;
+    $rootScope.starting = true;
     identityService.createDefaultWallet(function(err) {
       $scope.askForPin = 0;
+      $rootScope.starting = null;
 
       if (err) {
         var msg = err.toString();
@@ -143,10 +137,8 @@ angular.module('copayApp.controllers').controller('CreateProfileController', fun
     preconditions.checkArgument(_.isString(password));
 
     $rootScope.hideNavigation = false;
-    $rootScope.starting = true;
 
     identityService.create(emailOrUsername, password, function(err) {
-      $rootScope.starting = null;
       $scope.error = null;
       if (err) {
         var msg = err.toString();
@@ -162,6 +154,7 @@ angular.module('copayApp.controllers').controller('CreateProfileController', fun
       } else {
         // mobile
         if ($scope.isMobile) {
+          $rootScope.starting = null;
           _credentials = {
             email: emailOrUsername,
             password: password,
@@ -206,6 +199,7 @@ angular.module('copayApp.controllers').controller('CreateProfileController', fun
 
     $scope.saveSettings(function(err) {
       preconditions.checkState(!err, err);
+      $rootScope.starting = true;
 
       $scope._doCreateProfile($scope.userOrEmail, form.password.$modelValue, function(err) {
         $timeout(function() {
diff --git a/js/services/identityService.js b/js/services/identityService.js
index 7429c3727..9ae9c6983 100644
--- a/js/services/identityService.js
+++ b/js/services/identityService.js
@@ -323,7 +323,9 @@ angular.module('copayApp.services')
       if ($rootScope.iden) {
         $rootScope.signingOut = true;
         $rootScope.iden.close(function() { // Will trigger 'closed'
-          $rootScope.signingOut = false;
+          $timeout(function() {
+            $rootScope.signingOut = null;
+          }, 100);
         }); // Will trigger 'closed'
       }
     };
diff --git a/views/createProfile.html b/views/createProfile.html
index 443803d91..fa970ea3d 100644
--- a/views/createProfile.html
+++ b/views/createProfile.html
@@ -205,39 +205,38 @@
       <img src="img/clipo-pin.png" alt="clipo" width="380">
     </div>
 
-    <div class="box-notification" ng-show="error  && !passwordStrength">
+    <div class="box-notification" ng-show="error && !passwordStrength">
       <div class="box-icon error">
         <i class="fi-x size-24"></i>
       </div>
       <span class="text-warning size-14">
-          {{error|translate}}
-        </span>
+        {{error|translate}}
+      </span>
     </div>
 
-    <form name="setPinForm" novalidate>
+    <form name="setPinForm" ng-model="setPinForm" novalidate>
       <div class="input" ng-if="askForPin == 1">
         <input id="newpin" type="tel" ng-model="newpin" class="form-control" ng-maxlength="4" ng-minlength="4" maxlength="4" ng-pattern="/^[0-9]{1,4}$/" placeholder="PIN" name="newpin" required show-focus="askForPin == 1 && !isWindowsPhoneApp"  ng-focus="formFocus()">
-          <i class="icon-locked" ng-hide="isWindowsPhoneApp"></i>
+        <i class="icon-locked" ng-hide="isWindowsPhoneApp"></i>
       </div>
       <div class="input" ng-if="askForPin == 2">
-          <input id="repeatpin" type="tel" ng-model="repeatpin" class="form-control" ng-maxlength="4" ng-minlength="4" maxlength="4" ng-pattern="/^[0-9]{1,4}$/" placeholder="Confirm your PIN" name="repeatpin" required show-focus="askForPin == 2 && !isWindowsPhoneApp" ng-focus="formFocus()">
-          <i class="icon-locked" ng-hide="isWindowsPhoneApp"></i>
+        <input id="repeatpin" type="tel" ng-model="repeatpin" class="form-control" ng-maxlength="4" ng-minlength="4" maxlength="4" ng-pattern="/^[0-9]{1,4}$/" placeholder="Confirm your PIN" name="repeatpin" required show-focus="askForPin == 2 && !isWindowsPhoneApp" ng-focus="formFocus()">
+        <i class="icon-locked"></i>
       </div>
-        <div class="box-setup-footer row collapse">
-            <div class="large-6 medium-6 small-6 columns">
-                <a class="button secondary radius m0" ng-click="createDefaultWallet()">
-                    <span translate>Skip</span>
-                </a>
-            </div>
-            <div class="large-6 medium-6 small-6 columns text-right">
-                <button translate ng-click="createPin(setPinForm.repeatpin.$modelValue)" class="button primary radius expand m0"
-                        ng-disabled="setPinForm.$invalid || error">
-                    OK
-                </button>
-            </div>
+      <div class="box-setup-footer row collapse">
+        <div class="large-6 medium-6 small-6 columns">
+          <a class="button secondary radius m0" ng-click="createDefaultWallet()">
+            <span translate>Skip</span>
+          </a>
         </div>
-
-</form>
+        <div class="large-6 medium-6 small-6 columns text-right">
+          <button translate ng-click="createPin(setPinForm.repeatpin.$modelValue)" class="button primary radius expand m0"
+            ng-disabled="setPinForm.$invalid || error">
+            OK
+          </button>
+        </div>
+      </div>
+    </form>
   </div>
 </div>