Problema em mostra Status

[Jack Oliveira 1]

Olá pessoal boa noite

estou tendo um problema em mostra os Status como Ativo

no código php á definir para ser salvo como Ativo EX: 'status' =>  'S',

ele grava de boa 

mais lista Inativo mesmo estando no banco o valor S

 

para me mostrar se esta Ativo e Inativo fiz desta forma

 

	<?php if ($ListarCatalogo['status'] == 'S'){ ?>
<span><a href="?DesativarStatus=<?php echo $ListarCatalogo['id']; ?>" title="Item Inativo (<?php echo $ListarCatalogo['titulo']; ?>)" data-toggle="tooltip"  class="btn btn-sm btn-danger"><i class="fa fa-ban"></i></a></span>
	<?php }else{ ?>
<span><a href="?AtivarStatus=<?php echo $ListarCatalogo['id']; ?>" title="Item Ativo ( <?php echo $ListarCatalogo['titulo']; ?>)" data-toggle="tooltip"  class="btn btn-sm btn-success"><i class="fa fa-check"></i></a></span>
<?php } ?>

Tentei usar da seguinte forma tbm mais mesmo assim fica só invertido 

o que não seria o caso quando salvo ele não define nada

<?php if ($ListarCatalogo['status'] !== 'S'){ ?>

 

Vou mostra o código aqui mais sei que o problemas não esta aqui.

só para que todos possa entender melhor

// Ativar
    if (isset($_GET['AtivarStatus'])) {
        $id = get('AtivarStatus');
        $AtualizaCliente = array(
            'status'      => 'S'
        );

        $Query = DBUpdate("catalogo",$AtualizaCliente,"id = '{$id}'");
        if ($Query != 0) {
            Redireciona('?sucesso');
        } elseif ($Query == 0) {
            Redireciona('?erro');
        }
    }

// Desativar
    if (isset($_GET['DesativarStatus'])) {
        $id = get('DesativarStatus');
        $AtualizaCliente = array(
            'status'      => 'N'
        );

        $Query = DBUpdate("catalogo",$AtualizaCliente,"id = '{$id}'");
        if ($Query != 0) {
            Redireciona('?sucesso');
        } else {
            Redireciona('?erro');
        }
    }	

Este é o que faz o cadastro esta funcionando normal

// Adicionar Catalogo
	if (isset($_GET['Adicionar'])) {
$Adicionar = array(
			'titulo' 	     => post('titulo'),
			'preco' 	     => post('preco'),
			'data' 		     => date('d/m/Y'),
			'status' 	     => 'S',
			'id_categoria'   => post('id_categoria'),
			'obs'            => post('obs'),
			'descricao'      => post('descricao'),
			'id_user'        => DadosSession('id'),
			'codigo_produto' => post('codigo_produto'),
			'imagem' 	     => $Imagem

		);

		$Query = DBCreate('catalogo', $Adicionar);

		if ($Query != 0) {
	        Redireciona('?sucesso');
	    } else {
	        Redireciona('?erro');
	    }
	}

Espero que possam entender 

[quimera 13]

1 - Bom, eu acho na minha opinião, que você deveria usar orientação a objeto, isso facilitaria na manutenção.

 

 

2 - Defina umas constantes para não usar strings em condicionais

   define("ACTIVE_STATUS" , "n");//o bom seria usar inteiros, é minha visão
   define("INACTIVE_STATUS" , "s");

e quando for salvar no banco de dados use a constante como valor, e para comparar use a constante também.

<?php if ($ListarCatalogo['status'] == ACTIVE_STATUS){ ?>

 

 

3 - Verifique se $ListarCatalogo['status'] não é um objeto, ex.: $ListarCatalogo->status.

 

 

4 - Use algum pacote de log

 

 

5 - Teste TUDO! Pois procedural é f***!

 

[Jack Oliveira 1]

3 horas atrás, quimera disse:

1 - Bom, eu acho na minha opinião, que você deveria usar orientação a objeto, isso facilitaria na manutenção.

 

 

2 - Defina umas constantes para não usar strings em condicionais

   define("ACTIVE_STATUS" , "n");//o bom seria usar inteiros, é minha visão
   define("INACTIVE_STATUS" , "s");

e quando for salvar no banco de dados use a constante como valor, e para comparar use a constante também.

<?php if ($ListarCatalogo['status'] == ACTIVE_STATUS){ ?>

 

 

3 - Verifique se $ListarCatalogo['status'] não é um objeto, ex.: $ListarCatalogo->status.

 

 

4 - Use algum pacote de log

 

 

5 - Teste TUDO! Pois procedural é f***!

 

Olá amigo fiz da maneira aí em cima mais deu certo não