Notice: You are browsing the documentation for PrestaShop 9, which is currently in development.
You might want to read the documentation for the current version, PrestaShop 8. Read the current version of this page
Consistency is important, even more so when writing open-source code, since the code belongs to millions of eyeballs, and bug-fixing relies on these teeming millions to actually locate bugs and understand how to solve them.
This is why, when writing anything for PrestaShop, be it a theme, a module or a core patch, you should strive to follow the following guidelines. They are the ones that PrestaShop’s developers adhere to, and following them is the surest way to have your code be elegantly integrated in PrestaShop.
In short, code consistency helps keeping the code readable and maintainable.
All files containing code MUST:
Make sure that classes, attributes and methods are properly documented using doc blocks, regardless of visibility. As a rule of thumb, consider that a developer should be able to understand a component and use it without even having to read the source code – signatures and doc blocks should suffice for most cases.
Pay particular attention to the following:
$content
might be obviously a string, but we need more context to understand what its purpose is.PHP files MUST follow the PSR-2 standard alongside Symfony standards.
PHP CS Fixer has been configured for the PrestaShop project to help developers to comply with these conventions.
You can run it using the following command:
php ./vendor/bin/php-cs-fixer fix
The prestashop specific configuration file can be found here. Also, you can also use the provided git pre-commit sample in order to make sure you never forget to make your code compliant!
Parameters and return values SHOULD be described using Phpdoc as documented by Phpstan, especially when the type cannot be strictly defined using PHP types (e.g. collections, structs, mixed types).
Example:
/**
* @var string[] Collection of IETF language tag (eg. "en-US")
*/
public $locales;
Particular attention should be paid to documenting arrays and array structures:
string[]
or SomeType[]
.array<int, string>
.array{'foo': int, "bar": string}
.All new PHP code should be strictly typed.
This means that all new methods must specify a type for all parameters as well as the return type. Similarly, all new classes except interfaces must enforce type strictness via declare
:
<?php
/** 2007-2020 PrestaShop SA and Contributors... */
declare(strict_types=1);
namespace Foo\Bar;
class MyClass
{
public function doStuff(string $foo, array $bar): void
{
}
}
Following Symfony conventions, method and class deprecations in PrestaShop must be noted by adding the appropriate Phpdoc as well as a deprecation error:
<?php
namespace PrestaShop\Awesome\Path;
@trigger_error(
sprintf(
'%s is deprecated since version 8.0 and will be removed in the next major version.',
MyClass::class
),
E_USER_DEPRECATED
);
/**
* @deprecated Since 8.0 and will be removed in the next major.
*/
class MyClass
{
/**
* @deprecated Since 8.0, use AnotherClass::someNewMethod() instead.
*/
public function someOldMethod()
{
@trigger_error(
sprintf(
'%s is deprecated since version 8.0. Use %s instead.',
__METHOD__,
AnotherClass::class . '::someNewMethod()'
),
E_USER_DEPRECATED
);
}
}
If you need to deprecate services, you can use the deprecated
key for the service:
awesome.path.myclass:
class: 'PrestaShop\Awesome\Path\MyClass'
deprecated: 'The "%service_id%" service is deprecated since 8.0 and will be removed in next major.'
public: true
Javascript files MUST follow the Airbnb Javascript style guide.
You can run the linter to help you comply with these coding standards:
All admin-dev/themes/new-theme/js
files are coded in TypeScript. Classes and functions in .ts files must be strictly typed.
You are able to get global types in the admin-dev/themes/new-theme/js/types
folder using the @PSTypes
relative path and some types library are imported from npm using the @types
namespace and automatically imported by TypeScript.
npm run lint-fix
HTML, CSS (Sass), Twig and Smarty files MUST follow the Mark Otto’s coding standards. Mark is the creator of the Bootstrap framework.
To help developers to comply with these conventions, Stylelint, a stylesheet linter, has been configured in the PrestaShop project. You can find the configuration file on this repository.
Same as if you want to compile assets, you need NodeJS 20.x and NPM 8.x to run Stylelint.
You can run the linter like this:
npm run scss-lint
You can fix auto-fixable errors using this command:
npm run scss-fix
All PrestaShop files MUST start with the PrestaShop license block:
/**
* Copyright since 2007 PrestaShop SA and Contributors
* PrestaShop is an International Registered Trademark & Property of PrestaShop SA
*
* NOTICE OF LICENSE
*
* This source file is subject to the Open Software License (OSL 3.0)
* that is bundled with this package in the file LICENSE.md.
* It is also available through the world-wide-web at this URL:
* https://opensource.org/licenses/OSL-3.0
* If you did not receive a copy of the license and are unable to
* obtain it through the world-wide-web, please send an email
* to [email protected] so we can send you a copy immediately.
*
* DISCLAIMER
*
* Do not edit or add to this file if you wish to upgrade PrestaShop to newer
* versions in the future. If you wish to customize PrestaShop for your
* needs please refer to https://devdocs.prestashop-project.org/ for more information.
*
* @author PrestaShop SA and Contributors <[email protected]>
* @copyright Since 2007 PrestaShop SA and Contributors
* @license https://opensource.org/licenses/OSL-3.0 Open Software License (OSL 3.0)
*/
/**
* Copyright since 2007 PrestaShop SA and Contributors
* PrestaShop is an International Registered Trademark & Property of PrestaShop SA
*
* NOTICE OF LICENSE
*
* This source file is subject to the Academic Free License version 3.0
* that is bundled with this package in the file LICENSE.md.
* It is also available through the world-wide-web at this URL:
* https://opensource.org/licenses/AFL-3.0
* If you did not receive a copy of the license and are unable to
* obtain it through the world-wide-web, please send an email
* to [email protected] so we can send you a copy immediately.
*
* @author PrestaShop SA and Contributors <[email protected]>
* @copyright Since 2007 PrestaShop SA and Contributors
* @license https://opensource.org/licenses/AFL-3.0 Academic Free License version 3.0
*/