Fix the remaining compatibility issues

2 Warnungen gefunden. web/modules/contrib/permissions_dragcheck/permissions_dragcheck.module: ┌─────────┬──────┬──────────────────────────────────────────────────────────────┐ │ STATUS │ LINE │ MESSAGE │ ├─────────┼──────┼──────────────────────────────────────────────────────────────┤ │ Manuell │ 38 │ Call to deprecated function libraries_get_path(): Will be │ │ überpr� │ │ removed before a stable Drupal 8 release. Please use the new │ │ �fen │ │ library load and managment concepts described at: │ │ │ │ https://www.drupal.org/node/2170763 │ │ │ │ │ │ Manuell │ 68 │ Call to deprecated function libraries_get_path(): Will be │ │ überpr� │ │ removed before a stable Drupal 8 release. Please use the new │ │ �fen │ │ library load and managment concepts described at: │ │ │ │ https://www.drupal.org/node/2170763 │ │ │ │ │ └─────────┴──────┴──────────────────────────────────────────────────────────────┘

Comments & Activities

Issue created by @Grevil
Status changed to Needs work about 2 years ago12:06pm 27 March 2023
Comment about 2 years ago →
🇩🇪Germany Anybody Porta Westfalica
Thx @Grevil, could you please provide a MR with fixes? Thanks!!
@grevil opened merge request.
Status changed to Needs review about 2 years ago12:52pm 27 March 2023
Comment about 2 years ago →
🇩🇪Germany Grevil
Done! In my understanding, we do not need to implement the "hook_library_info_build" as it is only required for libraries, which need dynamic runtime information, but in this case there are none. Please correct me if I am wrong (Also fixed phpcs issues).
Comment about 2 years ago →
🇮🇹Italy apaderno Brescia, 🇮🇹
Assigned to Grevil
Status changed to Needs work about 2 years ago1:10pm 27 March 2023
Comment about 2 years ago →
🇩🇪Germany Anybody Porta Westfalica
@Grevil: Yes, I agree. Looks like we also don't need permissions_dragcheck_library_info_alter()? Could you check that?

https://www.drupal.org/node/2201089 → confirms your comment in #4 I think.

Still to be 100% sure please test manually after merging into dev and before creating a new release.
If you ensure both libraries are loaded correctly on the permission administration pages in 📌 Add basic install / uninstall and library loading test Postponed , we're 100% safe.
Status changed to RTBC about 2 years ago1:40pm 27 March 2023
Comment about 2 years ago →
🇩🇪Germany Grevil
Ok, I'll commit this to dev and test everything inside 📌 Add basic install / uninstall and library loading test Postponed .
Status changed to Fixed about 2 years ago1:41pm 27 March 2023
Comment about 2 years ago →
🇩🇪Germany Grevil
Comment about 2 years ago →
System Message

Grevil → committed 99aad0e9 on 2.x
Issue #3350619: Fix remaining comatibility issues
First commit to issue fork.
Comment about 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

Fix the remaining compatibility issues

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities