db_sync_tool.recipes.typo3.check_configuration()

Complexity

Conditions

7

Size

Total Lines	52
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

#!/usr/bin/env python3
def check_configuration(client):
    """
    Checking remote TYPO3 database configuration
    :param client: String
    :return:
    """
    _path = system.config[client]['path']

    if 'LocalConfiguration' in _path:
        stdout = mode.run_command(
            helper.get_command(client, 'php') + ' -r "echo json_encode(include \'' +
            system.config[client][
                'path'] + '\');"',
            client,
            True
        )

        _db_config = parse_database_credentials(json.loads(stdout)['DB'])
    elif '.env' in _path:
        # Try to parse settings from .env file
        if 'db' not in system.config[client]:
            system.config[client]['db'] = {}

        _db_config = {
            'name': get_database_setting_from_env(client, system.config[client]['db'].get('name', 'TYPO3_CONF_VARS__DB__Connections__Default__dbname'), system.config[client]['path']),

This line is too long as per the coding-style (183/100).

            'host': get_database_setting_from_env(client, system.config[client]['db'].get('host', 'TYPO3_CONF_VARS__DB__Connections__Default__host'), system.config[client]['path']),

This line is too long as per the coding-style (181/100).

            'password': get_database_setting_from_env(client, system.config[client]['db'].get('password', 'TYPO3_CONF_VARS__DB__Connections__Default__password'), system.config[client]['path']),

This line is too long as per the coding-style (193/100).

            'port': get_database_setting_from_env(client, system.config[client]['db'].get('port', 'TYPO3_CONF_VARS__DB__Connections__Default__port'), system.config[client]['path'])

This line is too long as per the coding-style (180/100).

            if get_database_setting_from_env(client, system.config[client]['db'].get('port', 'TYPO3_CONF_VARS__DB__Connections__Default__port'), system.config[client]['path']) != '' else 3306,

Wrong continued indentation (add 8 spaces).

This line is too long as per the coding-style (192/100).

            'user': get_database_setting_from_env(client, system.config[client]['db'].get('user', 'TYPO3_CONF_VARS__DB__Connections__Default__user'), system.config[client]['path']),

This line is too long as per the coding-style (181/100).

        }
    elif 'AdditionalConfiguration.php' in _path:
        # Try to parse settings from AdditionalConfiguration.php file
        _db_config = {
            'name': get_database_setting_from_additional_configuration(client, 'dbname', system.config[client]['path']),

This line is too long as per the coding-style (120/100).

            'host': get_database_setting_from_additional_configuration(client, 'host', system.config[client]['path']),

This line is too long as per the coding-style (118/100).

            'password': get_database_setting_from_additional_configuration(client, 'password', system.config[client]['path']),

This line is too long as per the coding-style (126/100).

            'port': get_database_setting_from_additional_configuration(client, 'port', system.config[client]['path'])

This line is too long as per the coding-style (117/100).

            if get_database_setting_from_additional_configuration(client, 'port',

Wrong continued indentation (add 8 spaces).

                                    system.config[client]['path']) != '' else 3306,

Wrong continued indentation (add 30 spaces).

            'user': get_database_setting_from_additional_configuration(client, 'user', system.config[client]['path']),

This line is too long as per the coding-style (118/100).

        }
    else:
        sys.exit(
            output.message(
                output.Subject.ERROR,
                f'Can\'t extract database information from given path {system.config[client]["path"]}. Can only extract settings from the following files: LocalConfiguration.php, AdditionalConfiguration.php, .env',

This line is too long as per the coding-style (214/100).

                False
            )
        )

    system.config[client]['db'] = helper.clean_db_config(_db_config)

The variable _db_config does not seem to be defined for all execution paths.

    return helper.run_sed_command(client, f'"s/{name}=(.*).*$/\\1/p" {file}')

Final newline missing