Synchronize hook reference pages with PrestaShop 9.1 metadata#2131
Synchronize hook reference pages with PrestaShop 9.1 metadata#2131MAX-IT-Tech wants to merge 1 commit into
Conversation
MAX-IT-Tech
commented
Apr 13, 2026
MAX-IT-Tech
commented
| Questions | Answers |
|---|---|
| Branch? | 9.x |
| Description? | Synchronizes hook reference pages with current PrestaShop 9.1 hook metadata and call examples. Updates origin file paths, aliases normalization, and hook invocation snippets to match current sources. |
| Fixed ticket? | N/A |
… and call examples
kpodemski
left a comment
kpodemski
left a comment
There was a problem hiding this comment.
First of all, thank you for your work. It seems it was automated, with some manual adjustments, but there are quite a few issues we need to address.
- Many hook aliases were incorrectly modified, this is crucial to fix.
- Quite a few hooks require adjustment in order to indicate that there are placed in multiple files.
- Some hooks were adjusted from
8.0.xto8.2.x, but if they don't exist in9.1.x, it means the code has been moved or removed. - There's a big problem because type of some display hooks was modified, not sure what caused
displaySomethingto be treated as a type: action, maybe something to be fixed in prompt? - Some hooks that are rendered in Smarty were modified to include
;at the end, which also has to change;`; is not needed and likely produces an error/not needed character.
I hope you can adjust the prompt to make the required fixes. I haven't added comments to every file that requires changes. I highly suggest you submit the next changes in a separate commit, without "force pushing" :)
This page is crucial to our developer community, and we need to ensure it contains the correct information.
There was a problem hiding this comment.
The filename is wrong on this one. It should point to classes/controller/AdminController.php on 9.1.x branch
There was a problem hiding this comment.
The filename is wrong on this one. It should point to classes/controller/AdminController.php on 9.1.x branch
There was a problem hiding this comment.
This file doesn't exist anymore, and this hook is used in AdminController
| hookTitle: | ||
| files: | ||
| - | ||
| url: 'https://github.com/PrestaShop/PrestaShop/blob/8.0.x/src/Adapter/Product/AdminProductDataProvider.php' |
There was a problem hiding this comment.
To be deleted, hook doesn't exist anymore. There's actionAjaxDie inside classes/controller/Controller.php
| type: action | ||
| hookAliases: displayOrderConfirmation |
| file: classes/controller/FrontController.php | ||
| locations: | ||
| - 'front office' | ||
| type: display | ||
| type: action |
| type: action | ||
| hookAliases: displayPaymentReturn |
There was a problem hiding this comment.
wrong path, it is inside src/Adapter/Admin/LegacyBlockHelperSubscriber.php
There was a problem hiding this comment.
hooks is inside multiple files
|
Hello @MAX-IT-Tech I hope you're doing well! Do you plan to finish this contribution? |
Unfortunately, I haven't been able to complete the contribution yet (I'm sick and busy with my day job, so I just don't have the energy right now), but I'll try to do it as soon as I can. I'm just wondering what the best course of action is: should I leave everything as is for now and continue later, or should I cancel the unfinished project and create new PRs based on the updated version of the documentation, PrestaShop, and your comments? Could you please advise me on the best course of action? |
2 participants
