Fix the link since the page migration

[Touxten]

Questions	Answers
Description?	Fix the link from the page migration?
Type?	bug fix
BC breaks?	no
Deprecations?	no
Fixed ticket?	PrestaShop/PrestaShop#39185
Sponsor company	@Touxten
How to test?	Use Nginx ( not apache ) and test all the buttons on the module.

[Hlavtox]

$this->context->link->getAdminLink('AdminModules', true)

[Touxten]

It's done @Hlavtox

I don't quite understand this file, There are four calls with false as a parameter.

[Hlavtox]

@Touxten Thanks!

I think that it's false because the token is assigned via $helper->token = Tools::getAdminTokenLite('AdminModules');, that's why the submit URL doesn't need it.

[Touxten]

You are right

[SiraDIOP]

Qa by Dev

[tleon]

Why did you put that in QA by dev ? I think in the future we should have a reason why an issue is put in QA by dev

[Hlavtox]

Ping @PrestaShop/qa-functional @PrestaShop/committers for QA, this is required for this module to work on 9.0.x branch on NGINX.

[kpodemski]

@Hlavtox If I recall correctly @Touxten managed to have it fixed with Nginx configuration change, could you confirm @Touxten ?

[SiraDIOP]

Hello @Touxten,
Thanks for your PR, it's fine with me. I tested it by following the instructions on how to test it.

However, the CI is red. Does that block the PR?

[Quetzacoalt91]

However, the CI is red. Does that block the PR?

If PHPStan (latest) is the only one failing, then it is fine to merge.