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.

CommentFileSizeAuthor
#2 799178_user_default_pic.patch3.08 KBsivaji_ganesh_jojodae
user-20100514.patch1.31 KBSteveK

Comments

SteveK’s picture

SteveK commented
Version: 7.0-alpha4 » 7.x-dev

changing to dev release.

sivaji_ganesh_jojodae’s picture

sivaji_ganesh_jojodae commented
Status: Active » Needs review
StatusFileSize
new 799178_user_default_pic.patch3.08 KB

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.)

retester2010’s picture

retester2010 commented

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

Status: Needs review » Needs work

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

pillarsdotnet’s picture

pillarsdotnet commented

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

pillarsdotnet’s picture

pillarsdotnet commented
Version: 7.x-dev » 8.x-dev
Issue tags: +Needs backport to D7

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

muranod’s picture

muranod commented
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.

muranod’s picture

muranod commented

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

muranod’s picture

muranod commented

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.

pillarsdotnet’s picture

pillarsdotnet commented
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.

muranod’s picture

muranod commented

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

pillarsdotnet’s picture

pillarsdotnet commented
Status: Active » Closed (works as designed)