Jump to content

Несуществующий manufacturer_id


Recommended Posts

Пользователи 1.2...1.4, проверьте плиз у себя — что будет если в поиске по производителям указать id несуществующего производителя?

 

Например так: /manufacturer.php?id_manufacturer=100500

 

Как будет выглядеть страница с ошибкой?

Share this post


Link to post
Share on other sites

Преста 1.3: все ломается

 

Преста 1.4 полет нормальный:

Производителя не существует.

или

Выявлено ошибок: 1 ошибка :

Производителя не существует.

Share this post


Link to post
Share on other sites

Ага, значит это не у меня одного глюк :)

Значить чтобы в 1.2...1.3 ничего не ломалось, нужно в /supplier.php этот код

$object = new $className(intval($id), $cookie->id_lang);
if (!Validate::isLoadedObject($object))
{
$errors[] = Tools::displayError('object does not exist');
}

заменить на код

$object = new $className(intval($id), $cookie->id_lang);
if (!Validate::isLoadedObject($object))
{
include(dirname(__FILE__).'/header.php');
$errors[] = Tools::displayError('object does not exist');
}

  • Like 1

Share this post


Link to post
Share on other sites

Ага, значит это не у меня одного глюк :)

Значить чтобы в 1.2...1.3 ничего не ломалось, нужно в /supplier.php этот код

$object = new $className(intval($id), $cookie->id_lang);
if (!Validate::isLoadedObject($object))
{
$errors[] = Tools::displayError('object does not exist');
}

заменить на код

$object = new $className(intval($id), $cookie->id_lang);
if (!Validate::isLoadedObject($object))
{
include(dirname(__FILE__).'/header.php');
$errors[] = Tools::displayError('object does not exist');
}

Да точно)

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×
×
  • Create New...

Important Information

Cookies ensure the smooth running of our services. Using these, you accept the use of cookies. Learn More