Created MR for patch in #9
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → changed the visibility of the branch 3486349-update-php-version to hidden.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → created an issue.
Working as Expected, Please see attached screenshot.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
You're correct. By avoiding the use of '#type' => 'path', you enable users to enter relative paths such as /abc rather than requiring a complete URL like https://www.example.com/abc. Instead, using '#type' => 'textfield' provides greater flexibility and aligns more closely with Drupal's standard handling of internal paths.
Updated the RedirectAfterRegistrationConfigForm to implement proper dependency injection for the path validator service. Added validation for the destination field to ensure it is a valid internal path.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
Updated code with this below :
if (isset($form_copy['subject'])) {
$form['subject'] = $form_copy['subject'];
}The use of the @ symbol in PHP is generally discouraged because it suppresses errors, which can make debugging difficult. Instead, we should use proper checks like isset() or empty() to safely handle variables.
zaryab_drupal → made their first commit to this issue’s fork.
Updated the Core Version Requirement
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
Working as expected.
See attached screenshot.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
Updated Configuration
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → made their first commit to this issue’s fork.
Added readme.md file to the code base in accordance to readme.txt
Please review and suggest if any other change required,
Moving the issue to needs review.
zaryab_drupal → made their first commit to this issue’s fork.
zaryab_drupal → changed the visibility of the branch 3399956-tab-highlights-for to hidden.