angular.controller(ꞌMainCtrlꞌ) - Code Metrics - Inspection of "Fix locales" - nextcloud/passman-webextension - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bebc18...c39f44 )

by Sander

created 2017-05-07 11:39 UTC

angular.controller(ꞌMainCtrlꞌ) B

↳ Parent: js/ui/popup/controllers/main.js

Complexity

Conditions	1
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	1
c	2
b	0
f	0
nc	2
nop	4
dl	0
loc	98
rs	8.3352

How to fix Long Method

/* global API */

/**
 * Nextcloud - passman
 *
 * @copyright Copyright (c) 2016, Sander Brand ([email protected])
 * @copyright Copyright (c) 2016, Marcos Zuriaga Miguel ([email protected])
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

(function () {
    'use strict';

    /**
     * @ngdoc function
     * @name passmanApp.controller:MainCtrl
     * @description
     * # MainCtrl
     * Controller of the passmanApp
     */
    angular.module('passmanExtension')
        .controller('MainCtrl', ['$scope', 'Settings', '$location', '$rootScope', function ($scope, Settings, $window, $rootScope) {
            $scope.app = 'passman';
            var port = API.runtime.connect(null, {
                name: "PassmanCommunication"
            });

            var messageParser = function (message) {
                var e = message.split(':');

                switch (e[0]) {

                    case "credential_amount":
                        $scope.credential_amount = e[1];
                        $scope.refreshing_credentials = false;
                }

                $scope.$apply();
            };

            /**
             * Connect to the background service
             */
            var initApp = function () {
                port.onMessage.addListener(messageParser);
                API.runtime.sendMessage(API.runtime.id, {method: "getMasterPasswordSet"}).then(function (isPasswordSet) {
                    function redirectToPrompt() {
                        window.location = '#!/locked';
                        return;

                    }

                    //First check attributes
                    if (!isPasswordSet) {
                        redirectToPrompt();
                        return;
                    }

                    getActiveTab();
                    $scope.refreshing_credentials = true;
                    setTimeout(function () {
                        port.postMessage("credential_amount");
                    }, 500);
                });
            };

            $scope.refreshing_credentials = false;
            $scope.refresh = function () {
                $scope.refreshing_credentials = true;
                API.runtime.sendMessage(API.runtime.id, {method: "getCredentials"}).then(function () {
                    setTimeout(function () {
                        port.postMessage("credential_amount");
                    }, 2000);
                });
            };

            var getActiveTab = function (cb) {

                API.tabs.query({currentWindow: true, active: true}).then(function (tab) {
                    API.runtime.sendMessage(API.runtime.id, {
                        method: "getCredentialsByUrl",
                        args: [tab[0].url]
                    }).then(function (_logins) {
                        //var url = backgroundPage.processURL(tab.url, $rootScope.app_settings.ignoreProtocol, $rootScope.app_settings.ignoreSubdomain, $rootScope.app_settings.ignorePath);
                        $scope.found_credentials = _logins;
                        $scope.$apply();
                    });
                });
            };

            $scope.lockExtension = function () {
                API.runtime.sendMessage(API.runtime.id, {method: "setMasterPassword", args: {password: null}}).then(function () {
                    window.location = '#!/locked';
                });
            };

            API.runtime.sendMessage(API.runtime.id, {'method': 'getRuntimeSettings'}).then(function (settings) {

                $rootScope.app_settings = settings;
                if (!settings || Object.keys(settings).length === 0) {
                    window.location = '#!/setup';
                } else if (settings.hasOwnProperty('isInstalled')) {
                    window.location = '#!/locked';
                } else {
                    initApp();
                }
            });



            $scope.goto_settings = function () {
                window.location = '#!/settings';
            };

            $scope.goto_search = function () {
                window.location = '#!/search';
            };

            $scope.editCredential = function (credential) {
                window.location = '#!/edit/' + credential.guid;
            };
        }]);
}());



1			/* global API */
2
3			/**
4			* Nextcloud - passman
5			*
6			* @copyright Copyright (c) 2016, Sander Brand ([email protected])
7			* @copyright Copyright (c) 2016, Marcos Zuriaga Miguel ([email protected])
8			* @license GNU AGPL version 3 or any later version
9			*
10			* This program is free software: you can redistribute it and/or modify
11			* it under the terms of the GNU Affero General Public License as
12			* published by the Free Software Foundation, either version 3 of the
13			* License, or (at your option) any later version.
14			*
15			* This program is distributed in the hope that it will be useful,
16			* but WITHOUT ANY WARRANTY; without even the implied warranty of
17			* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
18			* GNU Affero General Public License for more details.
19			*
20			* You should have received a copy of the GNU Affero General Public License
21			* along with this program. If not, see <http://www.gnu.org/licenses/>.
22			*
23			*/
24
25			(function () {
26			'use strict';
27
28			/**
29			* @ngdoc function
30			* @name passmanApp.controller:MainCtrl
31			* @description
32			* # MainCtrl
33			* Controller of the passmanApp
34			*/
35			angular.module('passmanExtension')
36			.controller('MainCtrl', ['$scope', 'Settings', '$location', '$rootScope', function ($scope, Settings, $window, $rootScope) {
37			$scope.app = 'passman';
38			var port = API.runtime.connect(null, {
39			name: "PassmanCommunication"
40			});
41
42			var messageParser = function (message) {
43			var e = message.split(':');
44
45			switch (e[0]) {
			0 ignored issues – show Coding Style introduced 2017-02-09 16:38 UTC by Report Bug Copy Issue Report As per coding-style, `switch` statements should have a `default` case. Loading history...
46			case "credential_amount":
47			$scope.credential_amount = e[1];
48			$scope.refreshing_credentials = false;
49			}
50
51			$scope.$apply();
52			};
53
54			/**
55			* Connect to the background service
56			*/
57			var initApp = function () {
58			port.onMessage.addListener(messageParser);
59			API.runtime.sendMessage(API.runtime.id, {method: "getMasterPasswordSet"}).then(function (isPasswordSet) {
60			function redirectToPrompt() {
61			window.location = '#!/locked';
62			return;
			0 ignored issues – show Unused Code introduced 2017-02-09 16:38 UTC by Report Bug Copy Issue Report This return has no effect and can be removed. Loading history...
63			}
64
65			//First check attributes
66			if (!isPasswordSet) {
67			redirectToPrompt();
68			return;
69			}
70
71			getActiveTab();
72			$scope.refreshing_credentials = true;
73			setTimeout(function () {
74			port.postMessage("credential_amount");
75			}, 500);
76			});
77			};
78
79			$scope.refreshing_credentials = false;
80			$scope.refresh = function () {
81			$scope.refreshing_credentials = true;
82			API.runtime.sendMessage(API.runtime.id, {method: "getCredentials"}).then(function () {
83			setTimeout(function () {
84			port.postMessage("credential_amount");
85			}, 2000);
86			});
87			};
88
89			var getActiveTab = function (cb) {
			0 ignored issues – show Unused Code introduced 2017-02-09 16:38 UTC by Report Bug Copy Issue Report The parameter `cb` is not used and could be removed. This check looks for parameters in functions that are not used in the function body and are not followed by other parameters which are used inside the function. Loading history...
90			API.tabs.query({currentWindow: true, active: true}).then(function (tab) {
91			API.runtime.sendMessage(API.runtime.id, {
92			method: "getCredentialsByUrl",
93			args: [tab[0].url]
94			}).then(function (_logins) {
95			//var url = backgroundPage.processURL(tab.url, $rootScope.app_settings.ignoreProtocol, $rootScope.app_settings.ignoreSubdomain, $rootScope.app_settings.ignorePath);
96			$scope.found_credentials = _logins;
97			$scope.$apply();
98			});
99			});
100			};
101
102			$scope.lockExtension = function () {
103			API.runtime.sendMessage(API.runtime.id, {method: "setMasterPassword", args: {password: null}}).then(function () {
104			window.location = '#!/locked';
105			});
106			};
107
108			API.runtime.sendMessage(API.runtime.id, {'method': 'getRuntimeSettings'}).then(function (settings) {
109
110			$rootScope.app_settings = settings;
111			if (!settings \|\| Object.keys(settings).length === 0) {
112			window.location = '#!/setup';
113			} else if (settings.hasOwnProperty('isInstalled')) {
114			window.location = '#!/locked';
115			} else {
116			initApp();
117			}
118			});
119
120
121
122			$scope.goto_settings = function () {
123			window.location = '#!/settings';
124			};
125
126			$scope.goto_search = function () {
127			window.location = '#!/search';
128			};
129
130			$scope.editCredential = function (credential) {
131			window.location = '#!/edit/' + credential.guid;
132			};
133			}]);
134			}());
135
136

nextcloud / passman-webextension

Push — master ( bebc18...c39f44 )

angular.controller(ꞌMainCtrlꞌ) B

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like