When tested with Upgrade Status module, I see a number of warning messages.

CUSTOM PROJECTS
--------------------------------------------------------------------------------
Field Permissions
Scanned on Sat, 06/06/2020 - 17:44.

4 warnings found.

web/modules/contrib/field_permissions/src/Controller/FieldPermissionsController.
php:
┌──────────┬──────┬────────────────────────────────────────────────────────────┐
│  STATUS  │ LINE │                          MESSAGE                           │
├──────────┼──────┼────────────────────────────────────────────────────────────┤
│ Check    │ 98   │ Call to deprecated method getList() of class               │
│ manually │      │ Drupal\field_permissions\FieldPermissionsServiceInterface: │
│          │      │ This function will be removed before 8.x-2.0               │
│          │      │                                                            │
│ Check    │ 148  │ Call to deprecated method getPermissionsByRole() of class  │
│ manually │      │ Drupal\field_permissions\FieldPermissionsServiceInterface: │
│          │      │ This function will be removed before 8.x-2.0               │
│          │      │                                                            │
└──────────┴──────┴────────────────────────────────────────────────────────────┘

web/modules/contrib/field_permissions/src/Plugin/FieldPermissionType/CustomAcces
s.php:
┌──────────┬──────┬────────────────────────────────────────────────────────┐
│  STATUS  │ LINE │                        MESSAGE                         │
├──────────┼──────┼────────────────────────────────────────────────────────┤
│ Check    │ 106  │ Call to deprecated method getList() of class           │
│ manually │      │ Drupal\field_permissions\FieldPermissionsService: This │
│          │      │ function will be removed before 8.x-2.0                │
│          │      │                                                        │
└──────────┴──────┴────────────────────────────────────────────────────────┘

web/modules/contrib/field_permissions/tests/src/Functional/FieldPermissionsTestB
ase.php:
┌──────────┬──────┬────────────────────────────────────────────────────────┐
│  STATUS  │ LINE │                        MESSAGE                         │
├──────────┼──────┼────────────────────────────────────────────────────────┤
│ Check    │ 141  │ Call to deprecated method getAllPermissions() of class │
│ manually │      │ Drupal\field_permissions\FieldPermissionsService: This │
│          │      │ function will be removed before 8.x-2.0                │
│          │      │                                                        │
└──────────┴──────┴────────────────────────────────────────────────────────┘




Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MrPaulDriver created an issue. See original summary.

extect’s picture

Status: Active » Closed (works as designed)

These are no functions from Drupal core, but from Field Permissions itself. Thus, field permissions does not use any deprecated functions from Drupal core. It just tells you Field Permissions will remove these functions in a future 8.x.-2.0 release of the module in case you are using these functions in some custom code.
--> Field permissions is fully compatible with Drupal 9

jhedstrom’s picture

Status: Closed (works as designed) » Active

It would still be good to fix field permissions to not call its own deprecated methods I think.

dshumaker’s picture

Attaching patch to take out deprecation messages.

  • jhedstrom committed e53e2f4 on 8.x-1.x authored by dshumaker
    Issue #3148380 by dshumaker: Drupal 9 compatibility. Warning messages...
jhedstrom’s picture

Status: Active » Fixed

Thanks! I think it makes sense to remove these deprecation annotations as they were added with no clear plan for replacing them, etc. If cleanup of removing these and refactoring is still needed, that can be done in a new issue.

Status: Fixed » Closed (fixed)

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