Renaming "setDefaultOptions" to "configureOptions" should replace "parent::setDefaultOptions" calls in body

[perk11]

Otherwise, there is an endless recursion in configureOptions.

[umpirsky]

Nice catch @perk11, thanks for reporting. 👍

[TomasVotruba]

@perk11 Hey, I'd like to include this feature, but I never used it this way.
Could please you provide before/after fix code, so I could understand it better?

Thanks

[aivus]

@TomasVotruba I think author means next:

Before

...
public function setDefaultOptions(OptionsResolverInterface $resolver) {
    parent::setDefaultOptions($resolver);
}
...

After:

...
public function configureOptions(OptionsResolver $resolver) {
    // Parent call renamed as well
    parent::configureOptions($resolver);
}
...

[TomasVotruba]

I see, thanks. I'll try to figure out what can I do with that

[TomasVotruba]

Thanks @aivus

I've added this feature to @rectorphp

Before

https://github.com/RectorPHP/Rector/blob/19d323e00e6161b961e597809e471c3bff3c01fb/tests/Rector/Dynamic/MethodNameReplacerRector/wrong/wrong6.php.inc

After

https://github.com/RectorPHP/Rector/blob/19d323e00e6161b961e597809e471c3bff3c01fb/tests/Rector/Dynamic/MethodNameReplacerRector/correct/correct6.php.inc