Why does Inner Join return repeated values?

Question

Good night,

I have a query to two tables where I want the second table to return all data with the id of table 1 but only returns me a table value when there are more than one with the same id. I need to get back all the data from the establishments table and return me the data that is associated with the establishment id of the establishments table.

Code

if($_REQUEST['valida'] != "ok"){
    if($result_alterar = $conexao->prepare("SELECT * FROM estabelecimentos 
                                            INNER JOIN categorias_estabelecimentos ON categorias_estabelecimentos.estabelecimento_id = estabelecimentos.id
                                            WHERE estabelecimentos.id = :id ")){
        $result_alterar->bindValue(':id', $_POST['id'], PDO::PARAM_INT);
        $result_alterar->execute();
        if($result_alterar->rowCount() >0 ){
            $row_alterar = $result_alterar->fetchAll(PDO::FETCH_ASSOC);
            $_REQUEST = $row_alterar; // O REQUEST ASSUME OS VALORES DO REGISTO, ASSIM EVITAMOS TER QUE CRIAR UM FORMULÁRIO INDEPENDENTE PARA AS EDIÇÕES

        }
    }
}

Table categories_establishments

CREATE TABLE IF NOT EXISTS `categorias_estabelecimentos` (
`id` int(11) NOT NULL AUTO_INCREMENT,
`estabelecimento_id` int(11) NOT NULL,
`categoria_slug` varchar(250) CHARACTER SET utf8 COLLATE utf8_unicode_ci NOT NULL,
PRIMARY KEY (`id`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 AUTO_INCREMENT=311 ;

Table establishments

 CREATE TABLE IF NOT EXISTS `estabelecimentos` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `titulo` varchar(250) COLLATE utf8_unicode_ci NOT NULL,
  `slug` varchar(500) COLLATE utf8_unicode_ci NOT NULL,
  `link_facebook` varchar(250) COLLATE utf8_unicode_ci NOT NULL,
  `link_mapa` text COLLATE utf8_unicode_ci NOT NULL,
  `distritos` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `concelhos` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `morada` varchar(250) COLLATE utf8_unicode_ci NOT NULL,
  `contacto` varchar(250) COLLATE utf8_unicode_ci NOT NULL,
  `int_preco` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `link_site` varchar(255) COLLATE utf8_unicode_ci NOT NULL,
  `introducao` text COLLATE utf8_unicode_ci NOT NULL,
  `servicos` varchar(500) COLLATE utf8_unicode_ci NOT NULL,
  `descricao` varchar(500) COLLATE utf8_unicode_ci NOT NULL,
  `keywords` varchar(500) COLLATE utf8_unicode_ci NOT NULL,
  `keywords_pesquisa` varchar(500) COLLATE utf8_unicode_ci NOT NULL,
  `google_verification` varchar(500) COLLATE utf8_unicode_ci NOT NULL,
  `activo` tinyint(1) NOT NULL,
  `pos` bigint(20) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci AUTO_INCREMENT=211 ;

Var Dump for $_REQUEST variable

http://pastebin.com/sJRswhvR

Answer 1

You’re not just taking the first line?

$row_alterar = $result_alterar->fetch(PDO::FETCH_ASSOC);

What happens if you order it to return ALL lines (An Array will come in the case):

$row_alterar = $result_alterar->fetchAll(PDO::FETCH_ASSOC);

This returns all the values, the problem is that it may not solve in its use (From what I understand has a form involved).

In this case I imagine that the solution would be more in the line of linking an array with the categories to the single establishment. Correct?

Code suggestion:

if($_REQUEST['valida'] != "ok"){
    if($result_alterar = $conexao->prepare("SELECT * FROM estabelecimentos WHERE estabelecimentos.id = :id ")){
        $result_alterar->bindValue(':id', $_POST['id'], PDO::PARAM_INT);
        $result_alterar->execute();
        if($result_alterar->rowCount() >0 ){
            $row_alterar = $result_alterar->fetch(PDO::FETCH_ASSOC);

            // Puxamos as categorias relevantes para esse estabelecimento
            $result_categorias = $conexao->prepare("SELECT * FROM categorias_estabelecimentos WHERE estabelecimento_id = :id ")
            $result_categorias->bindValue(':id', $_POST['id'], PDO::PARAM_INT);
            $result_categorias->execute();

            // Colocamos a listagem com as categorias como um elemento do estabelecimento
            // Esse é um exemplo que não foi testado, não manjo tanto do seu ORM.
            $row_alterar['categorias_estabelecimentos'] = $result_categorias->fetchAll(PDO::FETCH_ASSOC);

            // Agora basicamente você tem um campo "categorias_estabelecimentos" que vai ter a listagem de categorias, em um único resultado com o estabelecimento que você precisa trabalhar.

            $_REQUEST = $row_alterar; // O REQUEST ASSUME OS VALORES DO REGISTO, ASSIM EVITAMOS TER QUE CRIAR UM FORMULÁRIO INDEPENDENTE PARA AS EDIÇÕES

        }
    }
}

Suggestion to print checkboxes (Not completely related but may be useful):

<?php foreach ($categorias_estabelecimentos as $cat) { ?>
    <input type="hidden" name="categorias_estabelecimentos[<?= $cat['id'] ?>]" value="0" />
    <input type="checkbox" name="categorias_estabelecimentos[<?= $cat['id'] ?>]" value="1" /> <?= $cat['categoria_slug'] ?>
<?php } ?>

Hidden is for, in the post, to have information that something has been unchecked. It’s the default used by frameworks like Rails and Angular. The suggestion is partly unrelated but I consider relevant because it deals with the peculiarities of dealing with arrays in editing screens, which is relevant to the issue in question.