Zły kod błędu w SQLException na bazie MySQL, wszystko w Mojavi + Creole Opcje

[mike]

Mam problem z pogranicza php i MySQL ale zdecydowałem się umieścić temat tutaj.

Zaczne od kodu:

AddQuestionAction.class.php

[PHP] pobierz, plaintext 
<?php
 
	// ...
 
	public function execute()
	{
 
		// Pobranie obiektów z kontekstu
		$objController = $this->getContext()->getController();
		$objRequest	= $this->getContext()->getRequest();
		// ---
 
		// Pobranie modeli
		$objModel = $objController->getModel( 'FAQ', 'AddQuestion' );
		// ---
 
		try
		{
 
			$objModel->addQuestion();
 
		}
		catch( SQLException $objException )
		{
 
			// Jakieś tam operacje
 
 
			return View::ERROR;
 
		}
 
		return View::SUCCESS;
 
	}
 
	// ...
 
?>
[PHP] pobierz, plaintext 

AddQuestionModel.class.php

[PHP] pobierz, plaintext 
<?php
 
	// ...
 
	public function addQuestion()
	{
 
		// Pobranie modeli
		$objModel = $this->objController->getGlobalModel( 'FAQ' );
		// ---
 
		// Pobranie parametrów żądania
		$strSectionName = $this->objRequest->getParameter( 'name' );
		$intLanguageId  = $this->objRequest->getParameter( 'languageId' );
		$strQuestion	= $this->objRequest->getParameter( 'question' );
		$strAnswer	  = $this->objRequest->getParameter( 'answer' );		
		// ---
 
		// Pobranie danych z modeli
		$intMaxCurrentPosition = $objModel->getLastPosition( $intLanguageId );
		// ---
 
		$objStatment = $this->objConnection->prepareStatement( 'INSERT INTO faq SET languageId  = ?, position = ?, question = ?, answer = ?' );
 
		$arrParams = array(
						   $intLanguageId,
						   ( $intMaxCurrentPosition + 1 ),
						   $strQuestion,
						   $strAnswer
						  );
		$intAffectedRows = $objStatment->executeUpdate( $arrParams );
 
		if( $intAffectedRows > 0 )
		{
 
			return true;
 
		}
 
		return false;
 
	}
 
	// ....
 
?>
[PHP] pobierz, plaintext 

Jak widać powyżej - banalna operacja. W Akcji pobieram model i wykonuje jego metodę. Szafa gra, a przynajmniej powinna (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Podam dodatkowo strukturę tabeli faq:

[SQL] pobierz, plaintext 
CREATE TABLE `faq` (
  `id` int(10) NOT NULL AUTO_INCREMENT,
  `languageId` int(2) NOT NULL DEFAULT '0',
  `position` int(10) NOT NULL DEFAULT '1',
  `question` varchar(255) NOT NULL DEFAULT '',
  `answer` text NOT NULL,
  PRIMARY KEY  (`id`),
  UNIQUE KEY `question` (`languageId`,`question`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;
[SQL] pobierz, plaintext 

I teraz problem. Na pola languageId + question jest założony klucz unikalny. Co powinno zaowocowac wyrzuconym wyjątkiem podczas próby dodania tych samych danych.
I owszem wyjątek leci, ale ...
... z kodem błędu 0 (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) !
A przecież zaburzenie unikalności klucza to kod błędu 1062 (jak się nie mylę).

Dlaczego tak się dzieje :?:

Dodatek:
MySQL 4.1.7
php 5.0.4
Abstrakt bazodanowy to Creole 1.0.9
Framework Mojavi 3.0.0 dev ( + moje zmiany ).

[nospor]

Linia 32: robisz sql'owy insert a wywolujesz metode update. To chyba coś nie tak jest (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . sprawdź i daj znać

edit po poście M4chu:
no faktycznie, przeczytalem manuala i widzę że się myliłem. Ale plama na honorze (IMG:http://forum.php.pl/style_emoticons/default/sadsmiley02.gif) Mój błąd

[M4chu]

Linia 32: robisz sql'owy insert a wywolujesz metode update. To chyba coś nie tak jest (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) . sprawdź i daj znać

Wszystko w porządku: w Creole do inserta i update'a i delete'a używamy executeUpdate - RTFM (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)
A co do błędu, to może właśnie znalazleś buga, bo wyglada na dobrze napisane.

[mike]

Znalazłem ...
... niestety.

Mówię niestety bo natura błędu bardzo mnie zmartwiła. Jest to duże niedociągnięcie twórców Creole'a.

Zerknijcie jak w klasie SQLException wywoływany jest konstruktor klasy nadrzędnej (klasy Exception):
SQLException.php

[PHP] pobierz, plaintext 
<?php
 
	// ...
 
	/**
	 * Constructs a SQLException.
	 * @param string $msg Error message
	 * @param string $native Native DB error message.
	 * @param string $userinfo More info, e.g. the SQL statement or the connection s
tring that caused the error.
	 */
	public function __construct($msg, $native = null, $userinfo = null)
	{
		parent::__construct($msg);
		if ($native !== null) {
			$this->setNativeError($native);
		}
		if ($userinfo !== null) {
			$this->setUserInfo($userinfo);
		}
	}
 
	// ...
 
?>
[PHP] pobierz, plaintext 

... no właśnie (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) Bez drugiego parametru, który to jest kodem błędu.

Jak tak można? I to jeszcze pisząc pod php5 :?:

Bardzo nieładnie, bardzo.

Będę musiał przepisać Creole'a, albo poszukać altenatywy. A wielka szkoda, bo poza brakiem debugera myślałem że nie ma wad.

[Vengeance]

A czy czasem to

[PHP] pobierz, plaintext 
<?php
 
		if ($native !== null) {
			$this->setNativeError($native);
		}
 
?>
[PHP] pobierz, plaintext 

nie odpowiada za późniejsze ustawienie ów błędu? Może autorzy przewidzieli pobieranie kodu błędu z innego argumentu/inną metodą ?

[mike]

Niestety.

Też się łudziłem, ale przejrzałem kod Creole'a pod tym kątem i to jest fakt. Smutny zresztą.

W SQLException z Creole'a nie ma kodów błędów.

(Ale nic nie szkodzi, uzupełniłem sobie Creole'a o to)