js/ui/popup/controllers/search.js - 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

js/ui/popup/controllers/search.js A

↳ Parent: Project

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	1	Features	0

Metric	Value
cc	1
c	2
b	1
f	0
nc	1
nop	0
dl	0
loc	60
rs	9.5555

1 Function

Rating	Name	Duplication	Size	Complexity
A	angular.controller(ꞌSearchCtrlꞌ)	0	48	1

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('SearchCtrl', ['$scope', function ($scope) {
            $scope.app = 'passman';
            var port = API.runtime.connect(null, {
                name: "PassmanCommunication"
            });

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

                switch (e[0]) {
                    default:

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

                $scope.$apply();
            };
            port.onMessage.addListener(messageParser);
            port.postMessage("credential_amount");


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

            $scope.found_credentials = false;
            $scope.searchText = '';
            $scope.search = function () {
                API.runtime.sendMessage(API.runtime.id, {'method': 'searchCredential', args: $scope.searchText }).then(function (result) {
                    $scope.found_credentials = result;
                    $scope.$apply();
                });
            };


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

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


        }]);
}());



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('SearchCtrl', ['$scope', function ($scope) {
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]) {
46			default:
			0 ignored issues – show Coding Style Comprehensibility introduced 2017-04-09 08:30 UTC by Report Bug Copy Issue Report The default case is not the last statement in this switch statement. For the sake of readability, you might want to move it to the end of the statement. Loading history...
47			case "credential_amount":
48			$scope.credential_amount = e[1];
49			$scope.refreshing_credentials = false;
50			}
51
52			$scope.$apply();
53			};
54			port.onMessage.addListener(messageParser);
55			port.postMessage("credential_amount");
56
57
58			$scope.lockExtension = function () {
59			API.runtime.sendMessage(API.runtime.id, {method: "setMasterPassword", args: {password: null}}).then(function () {
60			window.location = '#!/locked';
61			});
62			};
63
64			$scope.found_credentials = false;
65			$scope.searchText = '';
66			$scope.search = function () {
67			API.runtime.sendMessage(API.runtime.id, {'method': 'searchCredential', args: $scope.searchText }).then(function (result) {
68			$scope.found_credentials = result;
69			$scope.$apply();
70			});
71			};
72
73
74			$scope.goto_settings = function () {
75			window.location = '#!/settings';
76			};
77
78			$scope.goto_search = function () {
79			window.location = '#!/search';
80			};
81
82
83			}]);
84			}());
85
86

nextcloud / passman-webextension

Push — master ( bebc18...c39f44 )

js/ui/popup/controllers/search.js A

Complexity

Size

Duplication

Importance

1 Function

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like