Adrien Foulon

Results 434 comments of Adrien Foulon

Almost a fix for #11 but the package name shouldn't be changed

This PR has tons of unrelated changes to the fix Here is a clean PR fixing the same error https://github.com/Adri-Glez/filepond-plugin-pdf-preview/pull/12

Thanks, I checked and it was missing so I created it, but still the same issue even after clearing the cache I have a different root category, maybe that's related?...

Ah I got it to work, I had to resave any category to clear the cache ![image](https://github.com/PrestaShop/PrestaShop/assets/6115458/26464708-b996-4fd3-9bfe-d9133f96d15a) So yes seems the missing translation for the root category is indeed an...

I'm trying to have turbo run a long lived dev php server outside of the project in watch mode ``` "//#run:server": { "dependsOn": [ "protos#build" ], "cache": false, "interactive": false,...

Maybe a proper fix should be both #1060 and this, because it was previously encouraged to add properties to the connection in `onOpen` which would trigger the warning as well

I found a quick and dirty solution, in the constructor of `Relation` (beware that it breaks some other cases) ```php public function __construct(Builder $query, Model $parent) { $this->query = $query;...

Sorry, that PR doesn't address this issue at all, I was only talking about how using static causes this kind of issues You can reopen it

@macropay-solutions I already tried this solution but it breaks some special cases The withConstraints approach might be the easieast workaround to get into the core

It will fail on those kind of relations because the Relation constructor is called twice in there and so it restores constraints too early ```php public function price_without_key_in_aggregates() { return...