Skip to content
This repository has been archived by the owner on May 4, 2018. It is now read-only.

Problemas com o postgres #12

Open
wants to merge 45 commits into
base: master
Choose a base branch
from
Open

Problemas com o postgres #12

wants to merge 45 commits into from

Conversation

aweiand
Copy link

@aweiand aweiand commented Jan 3, 2014

Tive alguns problemas com a base postgre, para os quais fiz as seguintes alterações para que funcionasse sem problemas:

Alterações para aperfeiçoamento do postgre:

Adicionado teste de "is_numeric($data)" na linha 72 para poder trabalhar melhor com colunas que sejam inteiras e não utilizar "LIKE" no "WHERE"
Removidas as aspas "´" e "`" para funcionamento correto
Suprimida a opção "PDO::ATTR_AUTOCOMMIT => true" na linha 573 para correção de bug na conexão do PDO
As alterações que efetuei:

Adicionado teste de tipo para verificar se for inteiro e poder utilizar campos com nomes diferentes de id no get, sem prejuízos a consulta no postgresql
if (is_numeric($data))
$query[] = sprintf('WHERE %s = ?', $id);
else
$query[] = sprintf('WHERE %s LIKE ?', $id);

Bug na Classe/Driver que não permite trocar o tipo de autocommit
$options += array
(
\PDO::ATTR_AUTOCOMMIT => true,
);

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants