Add scopehint on the category edit form#27
Conversation
sprankhub
left a comment
There was a problem hiding this comment.
Thanks a lot for working on this! I commented two minor things, which I found :) I wouldn't consider them necessary, though.
|
Hi @avstudnitz, do you consider merging this feature or is it a no-go? |
| $this->registry = $registry; | ||
| $this->categoryRepository = $categoryRepository; | ||
| $this->escaper = $escaper; | ||
| } |
There was a problem hiding this comment.
hi @thedotwriter
I would use the "constructor property promotion" introduced in PHP 8.0.
the properties are declared directly in the constructor which is a more concise and clean way to define class properties.
Example:
class CategoryFieldPlugin
{
/**
* @param StoreManagerInterface $storeManager
* @param Registry $registry
* @param CategoryRepositoryInterface $categoryRepository
* @param Escaper $escaper
*/
public function __construct(
public readonly StoreManagerInterface $storeManager,
public readonly Registry $registry,
public readonly CategoryRepositoryInterface $categoryRepository,
public readonly Escaper $escaper,
) {}
// rest of code
There was a problem hiding this comment.
Hi, thanks for checking this PR. This code is indeed quite old now. I will consider updating it if I ever get a feedback from the repo owner : p (@avstudnitz)
There was a problem hiding this comment.
I see. Well done btw!
Hope that this is approved and merged soon.
Hi,
Here's a feature I really wanted from this module. I adapted the \AvS\ScopeHint\Plugin\ProductEavDataProviderPlugin for the category edit form. I tried to stay close to the module coding style.
Check it out and tell me what you think.
Cheers