From 8630a14ee89921989ecce6c62c75e3d48230ded6 Mon Sep 17 00:00:00 2001
From: Jared Tabor <jtabor@ansible.com>
Date: Thu, 28 Apr 2016 16:48:28 -0700
Subject: [PATCH 1/2] Changing license workflow for first time user

---
 awx/ui/client/src/app.js                      |  4 ++
 .../src/bread-crumb/bread-crumb.partial.html  |  4 +-
 .../src/license/checkLicense.factory.js       | 31 ++++++++++++----
 awx/ui/client/src/license/license.block.less  | 16 +++++++-
 .../client/src/license/license.controller.js  | 21 +++++++++--
 .../client/src/license/license.partial.html   | 37 +++++++++++++++----
 awx/ui/client/src/license/license.route.js    | 11 +++++-
 .../client/src/main-menu/main-menu.block.less |  4 ++
 .../src/main-menu/main-menu.partial.html      | 12 +++++-
 awx/ui/templates/ui/index.html                |  4 +-
 10 files changed, 118 insertions(+), 26 deletions(-)

diff --git a/awx/ui/client/src/app.js b/awx/ui/client/src/app.js
index d57b217fee..9bebd68d1c 100644
--- a/awx/ui/client/src/app.js
+++ b/awx/ui/client/src/app.js
@@ -755,6 +755,9 @@ var tower = angular.module('Tower', [
 
 
                 $rootScope.$on("$stateChangeStart", function (event, next, nextParams, prev) {
+                    if (next.name !== 'signOut'){
+                        CheckLicense.notify();
+                    }
                     $rootScope.$broadcast("closePermissionsModal");
                     // this line removes the query params attached to a route
                     if(prev && prev.$$route &&
@@ -847,6 +850,7 @@ var tower = angular.module('Tower', [
                             $rootScope.sessionTimer = timer;
                             $rootScope.$emit('OpenSocket');
                             pendoService.issuePendoIdentity();
+                            CheckLicense.notify();
                         });
                     }
                 }
diff --git a/awx/ui/client/src/bread-crumb/bread-crumb.partial.html b/awx/ui/client/src/bread-crumb/bread-crumb.partial.html
index 6cdae3acb2..679da65803 100644
--- a/awx/ui/client/src/bread-crumb/bread-crumb.partial.html
+++ b/awx/ui/client/src/bread-crumb/bread-crumb.partial.html
@@ -1,5 +1,5 @@
 <div id="bread_crumb" class="BreadCrumb" ng-class="{'is-loggedOut' : !$root.current_user.username}">
-    <div ncy-breadcrumb></div>
+    <div ng-if="!licenseMissing" ncy-breadcrumb></div>
     <div class="BreadCrumb-menuLink"
         id="bread_crumb_activity_stream"
         aw-tool-tip="View Activity Stream"
@@ -8,6 +8,7 @@
         data-container="body"
         ng-class="{'BreadCrumb-menuLinkActive' : activityStreamActive}"
         ng-if="showActivityStreamButton"
+        ng-hide= "licenseMissing"
         ng-click="openActivityStream()">
         <i class="BreadCrumb-menuLinkImage icon-activity-stream"
             alt="Activity Stream">
@@ -20,6 +21,7 @@
         data-placement="left"
         data-trigger="hover"
         data-container="body"
+        ng-hide="licenseMissing"
         ng-if="!showActivityStreamButton">
         <i class="BreadCrumb-menuLinkImage fa fa-tachometer"
             alt="Dashboard">
diff --git a/awx/ui/client/src/license/checkLicense.factory.js b/awx/ui/client/src/license/checkLicense.factory.js
index 02a3b7198c..bec9fb77fb 100644
--- a/awx/ui/client/src/license/checkLicense.factory.js
+++ b/awx/ui/client/src/license/checkLicense.factory.js
@@ -5,13 +5,15 @@
  *************************************************/
 
 export default
-	['$state', '$rootScope', 'Rest', 'GetBasePath', 'ProcessErrors', function($state, $rootScope, Rest, GetBasePath, ProcessErrors){
+	['$state', '$rootScope', 'Rest', 'GetBasePath', 'ProcessErrors', '$q',
+	function($state, $rootScope, Rest, GetBasePath, ProcessErrors, $q){
 			return {
 				get: function() {
 					var defaultUrl = GetBasePath('config');
 					Rest.setUrl(defaultUrl);
 					return Rest.get()
 						.success(function(res){
+							$rootScope.license_tested = true;
 							return res;
 						})
 	                    .error(function(res, status){
@@ -50,13 +52,26 @@ export default
 					 	return true;
 				},
 				notify: function(){
-					var self = this;
-					this.get()
-						.then(function(res){
-							if(self.valid(res.data.license_info) === false) {
-								$state.go('license');
-							}
-						});
+					var deferred = $q.defer(),
+						self = this;
+					if($rootScope.license_tested !== true){
+						this.get()
+							.then(function(res){
+								if(self.valid(res.data.license_info) === false) {
+									$rootScope.licenseMissing = true;
+									deferred.resolve();
+									$state.go('license');
+								}
+								else {
+									$rootScope.licenseMissing = false;
+								}
+							});
+					}
+					else{
+						deferred.resolve();
+					}
+
+					return deferred.promise;
 				}
 
 			};
diff --git a/awx/ui/client/src/license/license.block.less b/awx/ui/client/src/license/license.block.less
index becc149b7e..5ea5065d6e 100644
--- a/awx/ui/client/src/license/license.block.less
+++ b/awx/ui/client/src/license/license.block.less
@@ -32,7 +32,7 @@
 }
 .License-eula textarea{
 	width: 100%;
-	height: 300px;
+	height: 200px;
 }
 .License-field label{
 	width: 155px;
@@ -64,6 +64,20 @@
 .License-management{
 	.OnePlusTwo-right--panel(650px);
 }
+.License-management--missingLicense{
+	height: auto;
+}
+.License-downloadLicenseButton{
+	background-color: @default-link !important;
+	margin-bottom: 10px;
+	color:@default-bg;
+}
+.License-downloadLicenseButton:hover{
+	color:@default-bg !important;
+}
+.License-downloadLicenseButton:focus{
+	color:@default-bg !important;
+}
 .License-submit--container{
 	height: 33px;
 }
diff --git a/awx/ui/client/src/license/license.controller.js b/awx/ui/client/src/license/license.controller.js
index d81b44b85d..4b26604793 100644
--- a/awx/ui/client/src/license/license.controller.js
+++ b/awx/ui/client/src/license/license.controller.js
@@ -5,10 +5,10 @@
  *************************************************/
 
 export default
-	[	'Wait', '$state', '$scope', '$rootScope', '$location',
-	 'GetBasePath', 'Rest', 'ProcessErrors', 'CheckLicense', 'moment',
-	 function( Wait, $state, $scope, $rootScope, $location,
-	 	GetBasePath, Rest, ProcessErrors, CheckLicense, moment){
+	['Wait', '$state', '$scope', '$rootScope', '$location', 'GetBasePath',
+	'Rest', 'ProcessErrors', 'CheckLicense', 'moment','$window',
+	function( Wait, $state, $scope, $rootScope, $location, GetBasePath, Rest,
+		ProcessErrors, CheckLicense, moment, $window){
 	 	$scope.getKey = function(event){
 	 		// Mimic HTML5 spec, show filename
 	 		$scope.fileName = event.target.files[0].name;
@@ -35,6 +35,11 @@ export default
 	 	$scope.fakeClick = function(){
 	 		$('#License-file').click();
 	 	};
+
+		$scope.downloadLicense = function(){
+            $window.open('https://www.ansible.com/license', '_blank');
+        };
+
 		$scope.newLicense = {};
 		$scope.submit = function(){
 			Wait('start');
@@ -48,6 +53,13 @@ export default
 						$scope.success = false;
 						clearTimeout(successTimeout);
 					}, 4000);
+					if($rootScope.licenseMissing === true){
+						$rootScope.licenseMissing = false;
+						$state.go('dashboard');
+					}
+					else{
+						$rootScope.licenseMissing = false;
+					}
 			});
 		};
 	 	var calcDaysRemaining = function(ms){
@@ -62,6 +74,7 @@ export default
 	 	};
 	 	var init = function(){
 	 		$scope.fileName = "Please choose a file...";
+			$scope.title = $rootScope.licenseMissing ? "Tower License" : "License Management";
 	 		Wait('start');
 	 		CheckLicense.get()
 	 		.then(function(res){
diff --git a/awx/ui/client/src/license/license.partial.html b/awx/ui/client/src/license/license.partial.html
index 8f7b7b81f5..7d35cce503 100644
--- a/awx/ui/client/src/license/license.partial.html
+++ b/awx/ui/client/src/license/license.partial.html
@@ -1,5 +1,5 @@
 <div class="License-container">
-	<div class="License-details">
+	<div class="License-details" ng-if="!licenseMissing">
 		<div class="Panel">
 			<div class="License-titleText">Details</div>
 			<div class="License-fields">
@@ -69,11 +69,32 @@
 			<a href="https://www.ansible.com/renew" target="_blank"><button class="btn btn-default">Upgrade</button></a>
 		</div>
 	</div>
-	<div class="License-management">
+	<div class="License-management" ng-class="{'License-management--missingLicense' : licenseMissing}">
 		<div class="Panel">
-			<div class="License-titleText">License Management</div>
+			<div class="License-titleText">{{title}}</div>
 			<div class="License-body">
-				<p class="License-helperText">Choose your license file, agree to the End User License Agreement, and click submit.</p>
+				<div class="AddPermissions-directions" ng-if="licenseMissing">
+					<span class="AddPermissions-directionNumber">
+						1.
+					</span>
+					<span class="License-helperText">
+						Please click the button below to visit Ansible's website to get a Tower license key.
+					</span>
+				</div>
+
+					<button class="License-downloadLicenseButton btn" ng-if="licenseMissing" ng-click="downloadLicense()">
+						Request License
+					</button>
+
+				<div class="AddPermissions-directions">
+					<span class="AddPermissions-directionNumber" ng-if="licenseMissing">
+						2.
+					</span>
+					<span class="License-helperText">
+						Choose your license file, agree to the End User License Agreement, and click submit.
+					</span>
+				</div>
+
 				<form id="License-form" name="license">
 					<div class="License-subTitleText prepend-asterisk"> License File</div>
 					<div class="input-group License-file--container">
@@ -89,12 +110,12 @@
 					<div class="form-group">
 						<div class="checkbox">
 							<div class="License-details--label"><input type="checkbox" ng-model="newLicense.eula" required>  I agree to the End User License Agreement</div>
-							<div class="License-submit--container pull-right">
-								<span ng-show="success == true" class="License-greenText License-submit--success pull-left">Save successful!</span>
-								<button ng-click="submit()" class="btn btn-success pull-right" ng-disabled="newLicense.file.license_key == null || newLicense.eula == null">Submit</button>
-							</div>
 						</div>
 					</div>
+					<div class="License-submit--container">
+						<span ng-show="success == true" class="License-greenText License-submit--success pull-left">Save successful!</span>
+						<button ng-click="submit()" class="btn btn-success pull-right" ng-disabled="newLicense.file.license_key == null || newLicense.eula == null">Submit</button>
+					</div>
 				</form>
 			</div>
 		</div>
diff --git a/awx/ui/client/src/license/license.route.js b/awx/ui/client/src/license/license.route.js
index 190ec6a5c7..947820a6e1 100644
--- a/awx/ui/client/src/license/license.route.js
+++ b/awx/ui/client/src/license/license.route.js
@@ -15,5 +15,14 @@ export default {
 	ncyBreadcrumb: {
 		parent: 'setup',
 		label: 'LICENSE'
-	}
+	},
+	resolve: {
+		features: ['CheckLicense', '$rootScope',
+			function(CheckLicense, $rootScope) {
+				if($rootScope.licenseMissing === undefined){
+					return CheckLicense.notify();
+				}
+
+		}]
+	},
 };
diff --git a/awx/ui/client/src/main-menu/main-menu.block.less b/awx/ui/client/src/main-menu/main-menu.block.less
index 89bafa127a..a35a88cea0 100644
--- a/awx/ui/client/src/main-menu/main-menu.block.less
+++ b/awx/ui/client/src/main-menu/main-menu.block.less
@@ -17,6 +17,10 @@
     align-items: stretch;
 }
 
+.MainMenu--licenseMissing{
+    justify-content: flex-end;
+}
+
 .MainMenu-logo,
 .MainMenu-item {
     color: @menu-link;
diff --git a/awx/ui/client/src/main-menu/main-menu.partial.html b/awx/ui/client/src/main-menu/main-menu.partial.html
index 9b2b3ee7af..bead6337d4 100644
--- a/awx/ui/client/src/main-menu/main-menu.partial.html
+++ b/awx/ui/client/src/main-menu/main-menu.partial.html
@@ -1,8 +1,9 @@
-<nav id="main_menu" class="MainMenu">
+<nav id="main_menu" class="MainMenu" ng-class="{'MainMenu--licenseMissing' : licenseMissing}">
     <!-- Menu logo item -->
     <a id="main_menu_logo"
         href="/#/"
         class="MainMenu-logo"
+        ng-if="!licenseMissing"
         ng-class="{'is-loggedOut' : !$root.current_user.username}">
         <img class="MainMenu-logoImage"
             ng-src="/static/assets/tower-logo-header.svg">
@@ -88,6 +89,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--left"
         id="main_menu_projects_link"
         href="/#/projects"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('projects'), 'is-loggedOut' : !$root.current_user.username}">
         <span class="MainMenu-itemText">
             PROJECTS
@@ -96,6 +98,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--left"
         id="main_menu_inventories_link"
         href="/#/inventories"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('inventories'), 'is-loggedOut' : !$root.current_user.username}">
         <span class="MainMenu-itemText">
             INVENTORIES
@@ -104,6 +107,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--left"
         id="main_menu_job_templates_link"
         href="/#/job_templates"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('jobTemplates'), 'is-loggedOut' : !$root.current_user.username}">
         <span class="MainMenu-itemText">
             JOB TEMPLATES
@@ -112,6 +116,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--left MainMenu-item--lastLeft"
         id="main_menu_jobs_link"
         href="/#/jobs"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('jobs'), 'is-loggedOut' : !$root.current_user.username}">
         <span class="MainMenu-itemText">
             JOBS
@@ -120,6 +125,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--user MainMenu-item--right"
         id="main_menu_current_user_link"
         ng-href="/#/users/{{ $root.current_user.id }}"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('users.edit'), 'is-loggedOut' : !$root.current_user.username}"
         aw-tool-tip="{{currentUserTip}}"
         aw-tip-watch="currentUserTip"
@@ -136,6 +142,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--right"
         id="main_menu_setup_link"
         ng-href="/#/setup"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('setup'), 'is-loggedOut' : !$root.current_user.username}"
         aw-tool-tip="Settings"
         data-placement="bottom"
@@ -148,6 +155,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--right"
         id="main_menu_portal_link"
         ng-href="/#/portal"
+        ng-if="!licenseMissing"
         ng-class="{'is-currentRoute' : isCurrentState('portalMode'), 'is-loggedOut' : !$root.current_user.username}"
         aw-tool-tip="Portal Mode"
         data-placement="bottom"
@@ -160,6 +168,7 @@
     <a class="MainMenu-item MainMenu-item--notMobile MainMenu-item--right"
         id="main_menu_docs_link"
         ng-href="http://docs.ansible.com/ansible-tower/"
+        ng-if="!licenseMissing"
         ng-class="{'is-loggedOut' : !$root.current_user.username}"
         aw-tool-tip="View Documentation"
         data-placement="bottom"
@@ -204,6 +213,7 @@
     <button
         id="main_menu_mobile_toggle_button"
         class="MainMenu-toggle"
+        ng-if="!licenseMissing"
         ng-class="{'is-active': !isHiddenOnMobile, 'is-loggedOut' : !$root.current_user.username}"
         ng-click="toggleMenu()">
         <i class="fa fa-bars MainMenu-toggleImage"></i>
diff --git a/awx/ui/templates/ui/index.html b/awx/ui/templates/ui/index.html
index 678edad92c..446c6d1798 100644
--- a/awx/ui/templates/ui/index.html
+++ b/awx/ui/templates/ui/index.html
@@ -171,7 +171,7 @@
             <div style="padding-bottom:15px;">For facts collected older than the time period specified,
                     save one fact scan (snapshot) per time window (frequency).
                     For example, facts older than 30 days are purged, while one
-                    weekly fact scan is kept.<br> <br>  
+                    weekly fact scan is kept.<br> <br>
 
                     CAUTION: Setting both numerical variables to "0" will delete all facts.<br><br>
             </div>
@@ -198,7 +198,7 @@
                     <label for="description">
     		                  <span class="label-text">
                                   Select a frequency for snapshot retention
-                             </span> 
+                             </span>
                     </label>
                     <div class="row">
                         <div class="col-xs-4">

From e0d181110ab7011491d4dccdb9d6a288e953f38f Mon Sep 17 00:00:00 2001
From: Jared Tabor <jtabor@ansible.com>
Date: Mon, 2 May 2016 13:42:26 -0700
Subject: [PATCH 2/2] cleaning up jshint errors

---
 .../manage-hosts/manage-hosts-add.controller.js  | 14 +++++++-------
 .../manage-hosts/manage-hosts-edit.controller.js | 16 ++++++++--------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-add.controller.js b/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-add.controller.js
index 0cad2dc9fa..620a68f1ac 100644
--- a/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-add.controller.js
+++ b/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-add.controller.js
@@ -17,7 +17,7 @@
         $scope.toggleEnabled = function(){
             $scope.host.enabled = !$scope.host.enabled;
         };
-        $scope.formSave = function(){;
+        $scope.formSave = function(){
             var params = {
                 variables: $scope.extraVars === '---' || $scope.extraVars === '{}' ? null : $scope.extraVars,
                 name: $scope.name,
@@ -25,18 +25,18 @@
                 enabled: $scope.host.enabled,
                 inventory: $stateParams.inventory_id
             };
-            ManageHostsService.post(params).then(function(res){
+            ManageHostsService.post(params).then(function(){
                 $state.go('^', null, {reload: true});
             });
         };
         var init = function(){
             $scope.host = {enabled: true};
             generator.inject(form, {mode: 'add', related: false, id: 'Inventory-hostManage--panel', scope: $scope});
-            ParseTypeChange({ 
-                scope: $scope, 
-                field_id: 'host_variables', 
-                variable: 'extraVars',  
+            ParseTypeChange({
+                scope: $scope,
+                field_id: 'host_variables',
+                variable: 'extraVars',
             });
         };
         init();
-    }];
\ No newline at end of file
+    }];
diff --git a/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-edit.controller.js b/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-edit.controller.js
index b8322fbdf2..de733124fa 100644
--- a/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-edit.controller.js
+++ b/awx/ui/client/src/inventories/manage/manage-hosts/manage-hosts-edit.controller.js
@@ -24,7 +24,7 @@
                 description: $scope.description,
                 enabled: $scope.host.enabled
             };
-            ManageHostsService.put(host).then(function(res){
+            ManageHostsService.put(host).then(function(){
                 $state.go('^', null, {reload: true});
             });
         };
@@ -34,12 +34,12 @@
             generator.inject(form, {mode: 'edit', related: false, id: 'Inventory-hostManage--panel', scope: $scope});
             $scope.extraVars = $scope.host.variables === '' ? '---' : $scope.host.variables;
             $scope.name = host.name;
-            $scope.description = host.description;          
-            ParseTypeChange({ 
-                scope: $scope, 
-                field_id: 'host_variables', 
-                variable: 'extraVars',  
+            $scope.description = host.description;
+            ParseTypeChange({
+                scope: $scope,
+                field_id: 'host_variables',
+                variable: 'extraVars',
             });
-        };           
+        };
         init();
-    }];
\ No newline at end of file
+    }];