Closed (fixed)
Project:
File (Field) Paths
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jul 2011 at 21:39 UTC
Updated:
17 Nov 2011 at 08:40 UTC
Jump to comment: Most recent file
Comments
Comment #1
Tim Jones Toronto commentedHi,
Had the same situation. Try after line 589 (e.g. on Dev version)
Locate where it has: $value = drupal_strtolower($value);
Just after this line add this code:
$value = str_replace(" ", "-", $value);Don't forget to enable: Convert to lower option in FFP.
Hope it helps!
Tim
Comment #2
dreadlocks1221 commentedworked well on retro update, thanks =^.^=
Comment #3
Tim Jones Toronto commentedCool :)
Comment #4
michelleWorked for me as well. Thanks!
Michelle
Comment #5
samtherobot commentedI'm not sure the thinking behind have a setting called "Pathauto" that never actually calls or references the module pathauto. Perhaps this just hasn't been finished yet.
I would think there needs to be a fairly simple block of code in the filefield_path_process_string function like this:
Comment #6
sweetchuckIf you confugere the pathauto to remove . (dot) then you will lose the file extensions.
If you confugere the pathauto to remove / (slash) then you will lose the directory hierarchy.
Comment #7
oierbravo commentedAdded code for cleaning up with pathauto.
It works like with the transliteration module, cleaning every part of the path separately so we don't lose directory hierarchy.
Comment #8
oierbravo commentedfixed patch
Comment #9
marcoka commentedi tested #8 it and it seems to work.
Comment #10
marcoka commentedthis does not seem the clean way to implement because in the latest dev there is already a check with a callback
line 557
'pathauto' => (module_exists('pathauto') && isset($settings['pathauto']) && $settings['pathauto'] && module_load_include('inc', 'pathauto')),
Comment #11
marcoka commentedComment #12
maximpodorov commentedThis revised patch is applied to latest dev version. It keeps slashes and dots and respects pathauto punctuation settings.
Comment #13
decipheredOther than a few minor coding standard issues, the code looks sound enough, will give it a spin shortly and commit if I'm happy with it.
Comment #14
decipheredCommitted to 7.x-1.x.
Thanks.
Comment #15.0
(not verified) commentedAdding info