Merge pull request #2063 from CodeNow/revert-ceab834b4e2517d1c1ee344a296951862c160037

Revert "Merge pull request #2051 from CodeNow/SAN-5804-implement-remo…

Author: thejsj

client/controllers/controllerInstances.js

@@ -18,7 +18,6 @@ function ControllerInstances(
   demoRepos,
   errs,
   eventTracking,
-  fetchGitHubRepoBranches,
   fetchInstances,
   fetchInstancesByPod,
   fetchRepoBranches,
@@ -269,7 +268,7 @@ function ControllerInstances(
     var branchName;
     var searchQuery = CIS.branchQuery.toLowerCase();
     return CIS.instanceBranches.filter(function (branch) {
-      branchName = branch.name.toLowerCase();
+      branchName = branch.attrs.name.toLowerCase();
       return branchName.includes(searchQuery);
     });
   };
@@ -301,8 +300,8 @@ function ControllerInstances(
     }, {});
     var instanceBranchName = instance.getBranchName();
     childInstances[instanceBranchName] = instanceBranchName;
-    var unbuiltBranches = branches.filter(function (branch) {
-      branchName = keypather.get(branch, 'name');
+    var unbuiltBranches = branches.models.filter(function (branch) {
+      branchName = keypather.get(branch, 'attrs.name');
       return !childInstances[branchName];
     });
     return unbuiltBranches;
@@ -312,13 +311,9 @@ function ControllerInstances(
     CIS.instanceBranches = null;
     CIS.poppedInstance = instance;
     loading('fetchingBranches', true);
-    var acv = instance.contextVersion.getMainAppCodeVersion();
-    var fullReponame = acv.attrs.repo.split('/');
-    var orgName = fullReponame[0];
-    var repoName = fullReponame[1];
-    return fetchGitHubRepoBranches(orgName, repoName)
+    return CIS.getAllBranches(instance)
       .then(function (branches) {
-        CIS.totalInstanceBranches = branches.length;
+        CIS.totalInstanceBranches = branches.models.length;
         CIS.instanceBranches = CIS.getUnbuiltBranches(instance, branches);
         loading('fetchingBranches', false);
       });
@@ -332,8 +327,8 @@ function ControllerInstances(
   };
 
   this.forkBranchFromInstance = function (branch, closePopover) {
-    var sha = branch.commit.sha;
-    var branchName = branch.name;
+    var sha = branch.attrs.commit.sha;
+    var branchName = branch.attrs.name;
     loading(branchName, true);
     loading('buildingForkedBranch', true);
     promisify(CIS.poppedInstance, 'fork')(branchName, sha)

client/directives/components/buildLogs/buildLogsDirective.js

@@ -4,7 +4,6 @@ require('app').directive('buildLogs', buildLogs);
 function buildLogs(
   $timeout,
   debounce,
-  featureFlags,
   moment,
   $interval,
   keypather
@@ -188,8 +187,7 @@ function buildLogs(
           $scope.timerExpired &&
           $scope.BLC.buildLogs.length === 0 &&
           !$scope.BLC.buildLogsRunning &&
-          $scope.BLC.buildStatus === 'failed' &&
-          featureFlags.flags.internalDebugging !== true
+          $scope.BLC.buildStatus === 'failed'
         );
       };
 

client/directives/components/newContainer/newContainerController.js

@@ -84,6 +84,7 @@ function NewContainerController(
     // Fetch all non-repo containres
     return fetchInstances({ githubUsername: 'HelloRunnable' })
       .then(function (servers) {
+        console.log('Fetched template servers', servers);
         NCC.templateServers = servers;
         loading('newContainerTemplates', false);
         return servers;

client/directives/instances/instance/branchMenuPopover/branchMenuPopoverView.jade

@@ -132,8 +132,8 @@
           )
             li.grid-block.align-center.list-item.popover-list-item(
               ng-class = "{\
-                'disabled': $root.isLoading['buildingForkedBranch'] && !$root.isLoading[branch.name],\
-                'active': $root.isLoading['buildingForkedBranch'] && $root.isLoading[branch.name]\
+                'disabled': $root.isLoading['buildingForkedBranch'] && !$root.isLoading[branch.attrs.name],\
+                'active': $root.isLoading['buildingForkedBranch'] && $root.isLoading[branch.attrs.name]\
               }"
               ng-repeat = "branch in CIS.getFilteredBranches()"
               ng-click = "CIS.forkBranchFromInstance(branch, POC.closePopover);"
@@ -143,13 +143,13 @@
                   xlink:href = "#icons-branch-alt"
                 )
               .grid-content.text-overflow(
-                title = "{{ branch.name }}"
-              ) {{ branch.name }}
+                title = "{{ branch.attrs.name }}"
+              ) {{ branch.attrs.name }}
               button.grid-content.shrink.btn.btn-xs.btn-icon.btn-add(
-                ng-if = "!$root.isLoading[branch.name]"
+                ng-if = "!$root.isLoading[branch.attrs.name]"
               ) Add
               .grid-content.shrink.spinner-wrapper.spinner-sm.spinner-green(
-                ng-if = "$root.isLoading[branch.name]"
+                ng-if = "$root.isLoading[branch.attrs.name]"
                 ng-include = "'spinner'"
               )
 

client/directives/instances/instanceList/DemoAddBranchController.js

@@ -21,16 +21,15 @@ function DemoAddBranchController(
 ) {
   var DBC = this;
 
-  function getBranchForPR (instance) {
-    var githubId = keypather.get(instance, 'attrs.contextVersion.owner.github') || currentOrg.github;
-    return promisify(githubId, 'fetchRepo')(instance.getRepoName())
+  function getBranchForPR () {
+    return promisify(currentOrg.github, 'fetchRepo')(DBC.instance.getRepoName())
       .then(function (repo) {
         return promisify(repo, 'fetchBranch')('dark-theme');
       })
       .then(function (branch) {
         var sha = branch.attrs.commit.sha;
         var branchName = branch.attrs.name;
-        return promisify(instance, 'fork')(branchName, sha);
+        return promisify(DBC.instance, 'fork')(branchName, sha);
       });
   }
 

client/services/demoFlowService.js

@@ -12,7 +12,6 @@ function demoFlowService(
   defaultContainerUrl,
   errs,
   featureFlags,
-  fetchGitHubRepoBranch,
   github,
   keypather,
   patchOrgMetadata,
@@ -134,15 +133,14 @@ function demoFlowService(
   }
 
   function forkNewInstance (instance) {
-    var acv = instance.contextVersion.getMainAppCodeVersion();
-    var fullReponame = acv.attrs.repo.split('/');
-    var orgName = fullReponame[0];
-    var repoName = fullReponame[1];
     addBranchListener();
-    return fetchGitHubRepoBranch(orgName, repoName, 'dark-theme')
+    return promisify(currentOrg.github, 'fetchRepo')(instance.getRepoName())
+      .then(function (repo) {
+        return promisify(repo, 'fetchBranch')('dark-theme');
+      })
       .then(function (branch) {
-        var sha = branch.commit.sha;
-        var branchName = branch.name;
+        var sha = branch.attrs.commit.sha;
+        var branchName = branch.attrs.name;
         return promisify(instance, 'fork')(branchName, sha);
       });
   }