xtc_check_categories_status

[Volker Strähle]

Warum wird für eine Kategorie mit Status 0 als Ergebnis 1 (true) zurückgegeben?

function xtc_check_categories_status($categories_id) {

	if (!$categories_id) {
		return 0;
  }

	$categorie_query = xtDBquery("SELECT parent_id,
                                       categories_status
                                  FROM ".TABLE_CATEGORIES."
                                 WHERE categories_id = '".(int)$categories_id."'
                                       ".CATEGORIES_CONDITIONS);
	
	$categorie_data = xtc_db_fetch_array($categorie_query, true);
	if ($categorie_data['categories_status'] == 0) {
		return 1;  // <<<<< RICHTIG ???
	} else {
		if ($categorie_data['parent_id'] != 0) {
			if (xtc_check_categories_status($categorie_data['parent_id']) >= 1) {
				return 1;
			}
		}
		return 0;
	}
}

Wäre nicht

function xtc_check_categories_status($categories_id) {

	if (!$categories_id) {
		return 0;
  }

	$categorie_query = xtDBquery("SELECT parent_id,
                                       categories_status
                                  FROM ".TABLE_CATEGORIES."
                                 WHERE categories_id = '".(int)$categories_id."'
                                       ".CATEGORIES_CONDITIONS);
	
	$categorie_data = xtc_db_fetch_array($categorie_query, true);
  if (($categorie_data['categories_status'] >= 1) && ($categorie_data['parent_id'] == 0)) {
		return 1;
	} else {
    if (($categorie_data['categories_status'] >= 1) && ($categorie_data['parent_id'] != 0)) {
			if (xtc_check_categories_status($categorie_data['parent_id']) >= 1) {
				return 1;
			}
		}
		return 0;
	}
}

richtiger?

[Gerhard Waldemair]

In 11703:

fix #1495