Avoid too many return statements within this method.

Open

                return $sets;

Avoid too many return statements within this method.

Open

        return true;

Function update_userlog_v114 has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

function update_userlog_v114(XoopsModule $module)
{
    $userlog    = Userlog::getInstance();
    $logsetsObj = $userlog->getHandler('setting')->getAll();
    $ret        = true;

• Read up
  Read up

Function xoops_module_update_userlog has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

function xoops_module_update_userlog(XoopsModule $module, $prev_version = null)
{
    if ($prev_version == round($module->getInfo('version') * 100, 2)) {
        $module->setErrors('You have the latest ' . $module->getInfo('name') . ' module (' . $module->getInfo('dirname') . ' version ' . $module->getInfo('version') . ') and update is not necessary');
//        print_r($module->getErrors());

• Read up
  Read up

Function update_userlog_v114 has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

function update_userlog_v114(XoopsModule $module)
{
    $userlog    = Userlog::getInstance();
    $logsetsObj = $userlog->getHandler('setting')->getAll();
    $ret        = true;

• Read up
  Read up

Function getModules has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function getModules($dirnames = [], $otherCriteria = null, $asObj = false)
    {
        // get all dirnames
        /** @var XoopsModuleHandler $moduleHandler */
        $moduleHandler = xoops_getHandler('module');

• Read up
  Read up

Function buildFileSelectEle has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function buildFileSelectEle($currentFile = [], $multi = false, $size = 3)
    {
        // $modversion['config'][$i]['options'] = array(_AM_USERLOG_FILE_WORKING=>'0',_AM_USERLOG_STATS_FILEALL=>'all');
        if (0 == count($currentFile) || '0' == $currentFile[0]) {
            $currentFile = $this->userlog->getWorkingFile();

• Read up
  Read up

Function exportLogsToCsv has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function exportLogsToCsv($logs, $headers, $csvNamePrefix = 'list_', $delimiter = ';')
    {
        $csvFile = $this->userlog->getConfig('logfilepath') . '/' . USERLOG_DIRNAME . '/export/csv/' . $csvNamePrefix . '_' . date('Y-m-d_H-i-s') . '.csv';
        // file create/open/write
        /** @var \XoopsFileHandler $fileHandler */

• Read up
  Read up

Function xoops_module_update_userlog has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

function xoops_module_update_userlog(XoopsModule $module, $prev_version = null)
{
    if ($prev_version == round($module->getInfo('version') * 100, 2)) {
        $module->setErrors('You have the latest ' . $module->getInfo('name') . ' module (' . $module->getInfo('dirname') . ' version ' . $module->getInfo('version') . ') and update is not necessary');
        print_r($module->getErrors());

• Read up
  Read up

Function userlog_views_show has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

function userlog_views_show($options)
{
    $loglogObj = UserlogLog::getInstance();
    $module    = [];
    if (!empty($options[1])) {

• Read up
  Read up

Function item has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function item($subscribe_from)
    {
        if (empty($subscribe_from)) {
            $script_arr             = [];
            $script_arr['topic_id'] = ['viewtopic.php'];

• Read up
  Read up

Function mergeFiles has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function mergeFiles($log_files = [], $mergeFileName = null)
    {
        $log_files = $this->parseFiles($log_files);
        if (0 == ($totalFiles = count($log_files))) {
            $this->setErrors(_AM_USERLOG_FILE_SELECT_ONE);

• Read up
  Read up

Function preventSubmitEmptyInput has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

function preventSubmitEmptyInput($prefix) {
    var eles = [];
    var inputs = document.getElementsByTagName("input");
    for (var i = 0; i < inputs.length; i++) {
        if (inputs[i].name.indexOf($prefix) == 0) {

• Read up
  Read up