+global\Profile::getCollapsedAttributes()

Complexity

Conditions	18
Paths	80

Size

Total Lines	54
Code Lines	37

Duplication

Lines	10
Ratio	18.52 %

Importance

Changes	1
Bugs	1	Features	0

<?php
    public function getCollapsedAttributes($eap = 0) {
        $attr = $this->getAttributes(0, $eap);
        $temp1 = [];
        foreach ($attr as $b) {
            $name = $b['name'];
            $temp1[] = $name;
            $level = $b['level'];
//            $S[$l] = $z[$l];

70% of this comment could be valid code. Did you maybe forget this after debugging?

            $value = $b['value'];
            if (!isset($temp[$name][$level])) {
                $temp[$name][$level] = [];

$temp was never initialized. Although not strictly required by PHP, it is generally a good practice to add $temp = array(); before regardless.

            }
            if ($b['flag'] == 'ML') {
                $v = unserialize($value);
                $value = [$v['lang'] => $v['content']];
            }
            $temp[$name][$level][] = $value;

The variable $temp does not seem to be defined for all execution paths leading up to this point.

            $flags[$name] = $b['flag'];

$flags was never initialized. Although not strictly required by PHP, it is generally a good practice to add $flags = array(); before regardless.

        }
        foreach ($temp1 as $name) {
            if ($flags[$name] == 'ML') {

The variable $flags does not seem to be defined for all execution paths leading up to this point.

                $S = [];
                if (isset($temp[$name]['Profile'])) {

This code seems to be duplicated across your project.

                    foreach ($temp[$name]['Profile'] as $z) {
                        foreach ($z as $l => $w) {
                            $S[$l] = $w;
                        }
                    }
                }
                if (!$S && isset($temp[$name]['IdP'])) {

The expression $S of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

This code seems to be duplicated across your project.

                    foreach ($temp[$name]['IdP'] as $z) {
                        foreach ($z as $l => $w) {
                            $S[$l] = $w;
                        }
                    }
                }
                $out[$name]['langs'] = $S;

$out was never initialized. Although not strictly required by PHP, it is generally a good practice to add $out = array(); before regardless.

                if (isset($S[$this->langIndex]) || isset($S['C'])) {
                    $out[$name][0] = (isset($S[$this->langIndex])) ? $S[$this->langIndex] : $S['C'];

The variable $out does not seem to be defined for all execution paths leading up to this point.

                }
            } else {
                if (isset($temp[$name]['Method'])) {
                    $out[$name] = $temp[$name]['Method'];
                }
                elseif (isset($temp[$name]['Profile'])) {
                    $out[$name] = $temp[$name]['Profile'];
                }
                else {
                    $out[$name] = $temp[$name]['IdP'];
                }
            }
        }
        return($out);
    }