Download & Extend

User default picture path incorrect when run through image styles

Project:Drupal core
Version:8.x-dev
Component:user.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (works as designed)
Issue tags:needs backport to D7

Issue Summary

When applying an image style to the user default picture, the returned URL filepath isn't structured properly (ex: pictures/user_default.jpg instead of public://pictures/user_default.jpg). I've also added better defined description text for the default user picture path text form item.

AttachmentSizeStatusTest resultOperations
user-20100514.patch1.31 KBIdleFAILED: [[SimpleTest]]: [MySQL] Setup environment: failed to drop checkout database.View details | Re-test

Comments

#1

Version:7.0-alpha4» 7.x-dev

changing to dev release.

#2

Status:active» needs review

Failed to apply patch, looks like it is not rolled from root of drupal file system

Hunk #1 FAILED at 377.
1 out of 1 hunk FAILED -- saving rejects to file user.patched/user.admin.inc.rej
patching file user.patched/user.module
Hunk #1 FAILED at 1321.
1 out of 1 hunk FAILED -- saving rejects to file user.patched/user.module.rej

Attached patch makes following changes on above patch,
- re-rolled from root directory
- adds $file_dir_path variable to avoid redundant function call file_directory_path()
- remove trailing white space from file user.module

(I would be easy if Default picture field in account settings page is file upload instead text field.)

AttachmentSizeStatusTest resultOperations
799178_user_default_pic.patch3.08 KBIdleFAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 799178_user_default_pic.patch.View details | Re-test

#3

#2: 799178_user_default_pic.patch queued for re-testing.

#4

Status:needs review» needs work

The last submitted patch, 799178_user_default_pic.patch, failed testing.

#5

I probably should have posted this here: http://drupal.org/node/935592#comment-3861338

#6

Version:7.x-dev» 8.x-dev

Not even sure if this bug still exists, but if it does, it needs to be fixed in 8.x first.

#7

Version:8.x-dev» 7.1

VERSION 7.10 and 7.11

I'm having the same issue, still running on my localhost. Problem is a double // (windows machine) in the path. Not yet sure where to fix it or if it's a bug or something I messed up myself..

Here's a working path from an older 7.10 install, followed by the broken path from a 7.11 (updated) install:

http://localhost/drupal/sites/default/files/styles/thumbnail/public/modu... (OLDER INSTALL)
http://localhost/drupal-7.10//sites/default/files/styles/thumbnail/publi... (BAD PATH - note the double // after drupal-7.10.

The requested page "/drupal-7.10//sites/default/files/styles/thumbnail/public/modules/image/sample.png?1327368160" could not be found.

#8

Found a similar problem here, but still can't fix it. I'm about to wipe everything I've done for the past month and a half and start over.
:-(

I even tried copying the cck folder from the older installation that was working fine and that made no difference, so I don't know where the problem lies.

http://drupal.org/node/1173294

#9

Sorry for the extra p.s. , but I just remembered that the problem first appeared after I installed and used the backup / migrate module. I've also seen that module referred to elsewhere in my searches regarding this problem, if that helps.

#10

Version:7.1» 8.x-dev
Status:needs work» active

Again; the bug needs to be fixed (or verified as absent) in 8.x before considering a patch to 7.x.

#11

FIXED!

Was not a bug. new USER ERROR.

from: http://drupal.org/node/1121774

Go to admin/config/media/file-system and make sure the paths there DO NOT have leading or trailing slashes!
It's that simple. Example: sites/default/files

Apologies if I had initially posted this in the wrong place. I am still working with v. 7-11.

Dan

#12

Status:active» closed (works as designed)