Replace AccountProxyInterface with interface implemented by User class

This is a good idea. Thank you.

I think rather than using UserInterface, we can use AccountInterface, which the User entity also implements via UserInterface.

RTBC +1

Deprecated function: str_starts_with(): Passing null to parameter #1 ($haystack) of type string is deprecated in Drupal\agreement\AgreementHandler->getAgreementByUserAndPath() (Zeile 154 in /mnt/web/mafo/html/modules/contrib/agreement/src/AgreementHandler.php)

Error occurs if:
$public_file_path = $this->settings->get('file_public_path')

$public_file_path is null. Seems php8.2 does not like it.

public function getAgreementByUserAndPath(AccountInterface $account, $path) {
    $agreement_types = $this->entityTypeManager->getStorage('agreement')->loadMultiple();

    $default_exceptions = [
      '/user/password',
      '/user/register',
      '/user/reset/*',
      '/user/login',
      '/user/logout',
      '/admin/config/people/agreement',
      '/admin/config/people/agreement/*',
      '/admin/config/people/agreement/manage/*',
    ];

    if ($public_file_path = $this->settings->get('file_public_path')) {
      $public_file_path = str_starts_with($public_file_path, '/')
        ? $public_file_path . '/*'
        : '/' . $public_file_path . '/*';
      // Adds the public file path to the default exceptions.
      $default_exceptions[] = $public_file_path;
    }
...

would fix it.

Okay, made a couple of fixes, and I think this is ready again. Thank you for the review.

RTBC +1

Merged. Thank you again for your review and opening up the issue.

I'll make a new alpha release shortly to help in your implementation.

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