drupal_get_path() error on theme-setting.php

Created on 21 June 2023, over 1 year ago
Updated 5 August 2023, over 1 year ago
🐛 Bug report
Status

Closed: outdated

Version

8.0

Component

Code

Created by

🇮🇳India sonam_sharma

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Comments & Activities

  • Issue created by @sonam_sharma
  • Issue was unassigned.
  • Status changed to Needs work over 1 year ago
  • 🇮🇹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
  • 🇮🇳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
  • 🇮🇳India ravis

    Thank you @all.
    The issue has been already fixed in version 10.x.x

Production build 0.71.5 2024