Function angeltypes_list_controller has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function angeltypes_list_controller() {
  global $privileges, $user;

  if (! in_array('angeltypes', $privileges))
    redirect('?');

• Read up
  Read up

Function User_shift_state_render has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function User_shift_state_render($user) {
  $upcoming_shifts = ShiftEntries_upcoming_for_user($user);
  if ($upcoming_shifts === false)
    return false;

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                  if ($ownshift['start'] >= $shift['start'] && $ownshift['start'] < $shift['end'] || $ownshift['end'] > $shift['start'] && $ownshift['end'] <= $shift['end'] || $ownshift['start'] < $shift['start'] && $ownshift['end'] > $shift['end']) {
                    $collides = true;
                    break;
                  }

Function user_driver_license_required_hint has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function user_driver_license_required_hint() {
  global $user;

  $angeltypes = User_angeltypes($user);
  if ($angeltypes === false)

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

                  if ($angeltype['count'] - count($entries) - $freeloader > 0) {
                    $inner_text = sprintf(ngettext("%d helper needed", "%d helpers needed", $angeltype['count'] - count($entries)), $angeltype['count'] - count($entries));
                    // is the shift still running or alternatively is the user shift admin?
                    $user_may_join_shift = true;

Avoid deeply nested control flow statements.

Open

          if ($event_id === false)
            engelsystem_error('Unable to load event type.');

Function shifts_json_export_controller has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function shifts_json_export_controller() {
  global $ical_shifts, $user;

  if (isset($_REQUEST['key']) && preg_match("/^[0-9a-f]{32}$/", $_REQUEST['key']))
    $key = $_REQUEST['key'];

• Read up
  Read up

Avoid deeply nested control flow statements.

Open

            if ($angeltype != null) {
             insert_by_room($id, $angeltype_id, $angeltype_count);
             $needed_angeltype_info[] = $angeltype['name'] . ": " . $angeltype_count;
            }

Function User_angeltypes_render has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

function User_angeltypes_render($user_angeltypes) {
  $output = array();
  foreach ($user_angeltypes as $angeltype) {
    $class = "";
    if ($angeltype['restricted'] == 1)

• Read up
  Read up

Method Settings_create has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

function Settings_create($event_name, $buildup_start_date, $event_start_date, $event_end_date, $teardown_end_date, $event_welcome_msg) {

Method update_rooms has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

function update_rooms($name, $from_pentabarf, $public, $number, $id, $event_id) {

Method Settings_update has 6 arguments (exceeds 4 allowed). Consider refactoring.

Open

function Settings_update($event_name, $buildup_start_date, $event_start_date, $event_end_date, $teardown_end_date, $event_welcome_msg) {

Consider simplifying this complex logical expression.

Open

    if ($wants_to_drive && ! $has_license_car && ! $has_license_3_5t_transporter && ! $has_license_7_5t_truck && ! $has_license_12_5t_truck && ! $has_license_forklift) {
      $ok = false;
      error(_("Please select at least one driving license."));
    }

Consider simplifying this complex logical expression.

Open

                  if ($ownshift['start'] >= $shift['start'] && $ownshift['start'] < $shift['end'] || $ownshift['end'] > $shift['start'] && $ownshift['end'] <= $shift['end'] || $ownshift['start'] < $shift['start'] && $ownshift['end'] > $shift['end']) {
                    $collides = true;
                    break;
                  }

Consider simplifying this complex logical expression.

Open

      if ($shift['shifttype_id'] != $tmp['shifttype_id'] || $shift['title'] != $tmp['title'] || $shift['start'] != $tmp['start'] || $shift['end'] != $tmp['end'] || $shift['RID'] != $tmp['RID'] || $shift['URL'] != $tmp['URL'])
        $shifts_updated[] = $shift;

Method form_checkbox has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

function form_checkbox($name, $label, $selected, $value = 'checked', $div = true) {

Method form_multi_checkboxes has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

function form_multi_checkboxes($names, $label, $items, $selected, $disabled = array()) {

Method ShiftType_edit_view has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

function ShiftType_edit_view($name, $angeltype_id, $angeltypes, $description, $shifttype_id) {

Method AngelType_edit_view has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

function AngelType_edit_view($name, $restricted, $description, $coordinator_mode, $requires_driver_license) {

Method progress_bar has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

function progress_bar($valuemin, $valuemax, $valuenow, $class = '', $content = '') {