Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Few FO hooks are not listed in the dev doc #1717

Closed
ElodieOS opened this issue Aug 25, 2023 · 7 comments
Closed

Few FO hooks are not listed in the dev doc #1717

ElodieOS opened this issue Aug 25, 2023 · 7 comments
Assignees

Comments

@ElodieOS
Copy link

I've found a few FO hooks that aren't listed in the devdoc. However, some of them exist and can be used in the transplant module list. I have noted this list :

  • displayProductButtons
  • graphEngine
  • gridEngine
  • paymentReturn
  • displayContactRightColomn
  • displayAdminOrderBottom
  • displayAdminOrderTabOrder
  • displayAdminOrderTabShip
  • displayAdminOrderTop
  • displayAdminOrdersListAfter
  • displayAdminStatsGraphEngine
  • displayAdminStatsGridEngine
  • displayAttributeForm
  • displayAttributeGroupPostProcess
  • displaybackOfficeFooter
  • displaybackOfficeHome
  • displaybackOfficeOrderActions
  • displayContentWrapperBottom
  • displayContentWrapperTop
  • displayFeatureValueForm
  • displayHeaderCategory
  • displayModuleConfigureExtraButtons (BO)
  • displayNav
  • displayPDFInvoice
  • displayProductExtraContent
  • displayWrapperBottom
  • displayWrapperTop
@Hlavtox
Copy link
Contributor

Hlavtox commented Aug 25, 2023

Good job @ElodieOS 🎉

Beware that some of them may be deprecated - paymentReturn for example?

I had a tool somewhere that scanned the code for hooks and compared it with devdocs and database, I will try to find it. :-)

@ElodieOS
Copy link
Author

Thank you @Hlavtox!

Oh it will be perfect if you have a tool like this yes! Indeed, I think some of them would be deprecated

@thomasnares
Copy link
Contributor

also Missing :

  • gridEngine
  • graphEngine

@PululuK
Copy link
Member

PululuK commented Sep 5, 2023

see #1540

@thomasnares
Copy link
Contributor

thomasnares commented Sep 26, 2023

Missing item Information Link
displayProductButtons created https://devdocs.prestashop-project.org/8/modules/concepts/hooks/list-of-hooks/displayproductbuttons/
graphEngine aliased to displayAdmonStatsGraphEngine #1733
paymentReturn renamed to displayPaymentReturn https://devdocs.prestashop-project.org/8/modules/concepts/hooks/list-of-hooks/displaypaymentreturn/
displayContactRightColomn typo in your list, it is displayContactRightColumn https://devdocs.prestashop-project.org/8/modules/concepts/hooks/list-of-hooks/displaycontactrightcolumn
displayAdminOrderBottom This one is listed in hook.xml but never rendered
displayAdminOrderTabOrder deprecated from 1.7.7
displayAdminOrderTabShip deprecated from 1.7.7
displayAdminOrderTop fixed #1733
displayAdminOrdersListAfter Unable to find any valid reference to this one
displayAdminStatsGraphEngine created #1733
displayAdminStatsGridEngine created #1733
displayAttributeForm created #1733
displayAttributeGroupPostProcess needs some work to be integrated (ps_faceted_search)
displaybackOfficeFooter deprecated from 1.7.7
displaybackOfficeHome deprecated from at least 1.6.x
displaybackOfficeOrderActions aliased to displayAdminOrderSide
displayContentWrapperBottom created
displayContentWrapperTop created
displayFeatureValueForm created #1733
displayHeaderCategory created #1733
displayModuleConfigureExtraButtons created #1733
displayNav use displayNav1 or displayNav2 for classic derived themes
displayPDFInvoice created
displayProductExtraContent created #1749
displayWrapperBottom created
displayWrapperTop created

@thomasnares
Copy link
Contributor

Related PR to fix missing displayProductButtons : PrestaShop/ps-docs-theme#45

@thomasnares
Copy link
Contributor

all done 👍

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

No branches or pull requests

4 participants