in #22336: Move all core Drupal files under a /core folder to improve usability and upgrades comment #299. quicksketch changed the chdir() in password-hash.sh to:

chdir('../..');

This is causing the script to go up two levels, simply out of Drupal dir, and causing password-hash.sh to stop working (includes paths not found). The correct chdir() is one-level only:

chdir('..');
CommentFileSizeAuthor
#1 1333192-1.patch380 bytesgood_man
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

good_man’s picture

good_man CreditAttribution: good_man commented
FileSize
1333192-1.patch380 bytes
good_man’s picture

good_man CreditAttribution: good_man commented
Status: Active » Needs review
xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm commented
Status: Needs review » Reviewed & tested by the community

Straightforward fix for a straightforward bug.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented

+1

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Fix looks fine and this is untestable. Committed/pushed to 8.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.