Creation of dynamic property Drupal\tfa\Plugin\Tfa\TfaTotp::$isValid is deprecated

Hi Mark,

Patch is good and applies cleanly. It's a simple and logical fix, so hopefully this will get accepted by the maintainer.

Thanks!

It fixes the problem, but this is not the correct way to fix it.

The isValid property is only ever used inside the protected validate() function in the TfaRecoveryCode, TfaHotp and TfaTotp plugins, and can be trivially simplified by the direct return of the boolean value instead of the assignment to the dynamic property followed by the return.

I think we can all agree that promoting an internal temporary value of the validate() function to be a property of the plugin base class is not the best way to solve this.

I was just assuming that the implementation was intentional, but I am happy to make the change as suggested!

We will use this patch in drupal 10 until it is officially released.

jcnventura → closed merge request !27

Automatically closed - issue fixed for 2 weeks with no activity.

Can we get this in next release?