This issue is created to separately track the D10 compatibility
so that the actions here do not interfere with the automated patches at
https://www.drupal.org/project/organigrams/issues/3297792
As part of this issue:
1. Will check the module compatibility with drupal 10 (before and after applying the auto-bot patch) using the Upgrade-statue module scan.
2. Will check the depreciated method(if any) after the patch is applied.
3. Will provide the patch to solve the errors if any for the module
4. Will finally check the module installed and working on the drupal 10 instance.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | drupal10-compatibility-3301253-6.patch | 2.71 KB | jaydev bhatt |
| #2 | Screenshot from 2022-08-02 16-04-23.png | 170.53 KB | akshaydalvi212 |
| #2 | Screenshot from 2022-08-02 16-01-45.png | 53.9 KB | akshaydalvi212 |
| #2 | Screenshot from 2022-08-02 14-29-19.png | 22.36 KB | akshaydalvi212 |
| #2 | Screenshot from 2022-08-02 14-27-38.png | 15.31 KB | akshaydalvi212 |
Comments
Comment #2
akshaydalvi212 commentedHello,
Thanks for the auto-bot patch.
as the patch is already committed in the branch and makes the module compatible with drupal 10.
The result of the scan by the Upgrade-statue module confirms the compatibilities with drupal 10.
Also, the module gets installed on drupal 10 smoothly.
while working with the module, the organigrams gets created just like taxonomy terms list
but when we view the organigram, ends in throwing error as below
Need to solve this error to make the module work on drupal 10.
Thanks and regards
Comment #3
reenaraghavan commentedComment #4
reenaraghavan commentedComment #5
jaydev bhatt commentedComment #6
jaydev bhatt commentedi have checked this module with Drupal 10
attaching a patch to remove/upgrade deprecated code to solve the error mention in the #2.
Steps to reproduce the error from #2
- install the module into drupal 10
- create an Organigrams from the Structure > taxonomy > Organigrams
- click on the option that says " View " from the tabs above
Please review the attached patch.
Thanks and regards.
Comment #8
patrickscheffer commentedSorry for the delay. I checked your patch and applied it to the 8.x-2.x branch. Could you verify that it works?
Comment #9
patrickscheffer commented