Function testMakensis has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    private function testMakensis()
    {
        if (!is_numeric(\config\ConfAssistant::NSIS_VERSION)) {
            $this->storeTestResult(\core\common\Entity::L_ERROR, "NSIS_VERSION needs to be numeric!");
            return;

• Read up
  Read up

Function longestNameSuffix has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function longestNameSuffix()
    {
        // for all configured server names, find the string that is the longest
        // suffix to all of them
        $longestSuffix = "";

• Read up
  Read up

Function writeInstaller has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function writeInstaller()
    {
        \core\common\Entity::intoThePotatoes();
        $this->prepareInstallerLang();
        $this->setupEapConfig();

• Read up
  Read up

Function processCertificate has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function processCertificate($cadata)
    {
        $pemBegin = strpos($cadata, "-----BEGIN CERTIFICATE-----");
        if ($pemBegin !== FALSE) {
            $pemEnd = strpos($cadata, "-----END CERTIFICATE-----") + 25;

• Read up
  Read up

Function __construct has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    public function __construct($idpObject, $deploymentIdRaw, $consortium = 'eduroam')
    {
        parent::__construct($idpObject, $deploymentIdRaw); // we now have access to our INST database handle and logging
        $this->entityOptionTable = "deployment_option";
        $this->entityIdColumn = "deployment_id";

• Read up
  Read up

Function __get_graphics_support has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    def __get_graphics_support(self) -> None:
        self.graphics = 'tty'
        if self.gui == 'tty':
            return
        if os.environ.get('DISPLAY') is None:

• Read up
  Read up

Function __init__ has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.

Open

    def __init__(self, silent: bool = False, username: str = '',
                 password: str = '', pfx_file: str = '', gui: str = '') -> None:
        self.graphics = ''
        self.username = username
        self.password = password

• Read up
  Read up

Method displaySilverbulletPropertyWidget has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function displaySilverbulletPropertyWidget(&$theProfile, $readonly, &$uiElements) {
    ?>
    <div style='padding-bottom:20px;'>
        <h2><?php echo $theProfile->name; ?></h2>
        <?php

Method update_table has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function update_table($table_name) {
        $table = 'view_'.$table_name;
        $tmp_table = 'tmp_'.$table_name;
        if ($this->tablesource[$table_name] == 'eduroam') {
            $this->db_local->select_db('monitor_copy');

Method combineLogo has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    protected function combineLogo($logos = NULL, $fedLogo = NULL)
    {
        // maximum size to which we want to resize the logos

        $maxSize = 120;

Method writeMessages has 35 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function writeMessages($file) {
        \core\common\Entity::intoThePotatoes();
        $messages = [
        'quit'=> _("Really quit?"),
        'username_prompt'=> _("enter your userid"),

Method eapBlock has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function eapBlock($caRefs)
    {
        $selectedEap = $this->selectedEap;
        $outerId = $this->determineOuterIdString();
        $eapPrettyprint = \core\common\EAP::eapDisplayName($selectedEap);

Method saveNetworkProfileSeparateHS has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function saveNetworkProfileSeparateHS($profileName, $network)
    {
        $out = '';
        if (!empty($network['ssid'])) {
            if ($this::separateSSIDprofiles === true && !empty($network['condition']) && $network['condition'] === 'locally_defined') {

Method generateNewInstaller has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function generateNewInstaller($device, $profile, $generatedFor, $openRoaming, $token, $password)
    {
        $this->loggerInstance->debug(5, "generateNewInstaller() - Enter");
        $this->loggerInstance->debug(5, "generateNewInstaller:openRoaming:$openRoaming\n");
        $factory = new DeviceFactory($device);

Method __construct has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function __construct()
    {
            
        if ( \config\ConfAssistant::eduPKI['testing'] === true ) {
            $this->locationRaCert = ROOT . "/config/SilverbulletClientCerts/edupki-test-ra.pem";

Method addCrltoCert has 34 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function addCrltoCert(&$cert) {
        $crlUrl = [];
        $returnresult = 0;
        if (!isset($cert['full_details']['extensions']['crlDistributionPoints'])) {
            return RADIUSTests::CERTPROB_NO_CDP;

Function row_filter has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function row_filter(tbody) {
    var linked = tbody.find('[id^="unlinked_ck_"]').is(':checked');
    var broken_cert = tbody.find('[id^="brokencert_ck_"]').is(':checked');
    var or_warn = tbody.find('[id^="or_ck_"]').is(':checked');
    var profile_warn = tbody.find('[id^="profile_ck_"]').is(':checked');

Method deleteObsoleteTempDirs has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function deleteObsoleteTempDirs() {
        $downloadsDirs = [
            'site_installers' => dirname(dirname(dirname(dirname(__FILE__)))) . "/var/installer_cache",
            'silverbullet' => dirname(dirname(dirname(dirname(__FILE__)))) . "/var/silverbullet"
        ];

Method saveCertificateFiles has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    final protected function saveCertificateFiles($format)
    {
        switch ($format) {
            case "der": // fall-thorugh, same treatment
            case "pem":

Method __construct has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function __construct($realm, $outerUsernameForChecks, $supportedEapTypes = [], $expectedServerNames = [], $expectedCABundle = []) {
        parent::__construct();

        $this->realm = $realm;
        $this->outerUsernameForChecks = $outerUsernameForChecks;