- Issue created by @sonam_sharma
- Issue was unassigned.
- Status changed to Needs work
over 1 year ago 11:03am 3 July 2023 - 🇮🇹Italy apaderno Brescia, 🇮🇹
The issue summary should always describe what should be fixed/changed. Neither the title nor screenshots are sufficient to describe what the issue is, even in the case a patch is provided.
In the case of a bug, the description should also list the steps necessary to reproduce the issue, starting from when the module/theme is installed. - Status changed to Needs review
over 1 year ago 8:11pm 5 August 2023 - 🇮🇳India Abh1shek
Status was not changed from needs work to needs review as there is patch provided. Changing status to Needs review and also reviewing this patch.
- 🇮🇳India Abh1shek
Hi @sonam_sharma, your patch applied cleanly.
I followed these steps:1. Took clone from branch 8.0.x in drupal 10.1.2
2. Installed thex theme.
3. Installed theme and set it as default.
4. Applied the patch.But error for undefine function is still there because you have made changes in theme-setting.php but the issue is occurring in edux.theme file.
I have created a patch. I replaced the deprecated function
drupal_get_path() with \Drupal::service('extension.list.theme')->getPath('edux'). Providing patch and screenshots.BEFORE
AFTER
Please review.
- Status changed to Closed: outdated
over 1 year ago 11:53pm 5 August 2023 - 🇮🇳India ravis
Thank you @all.
The issue has been already fixed in version 10.x.x