Problem/Motivation
IMHO users don't care where the files are relative to drupal, they just want to be able to use the image
"meta": {
"derivatives": {
"recipe_list_350_300": "/sites/default/files/styles/recipe_list_350_300/https/www.cookipedia.co.uk/wiki/images/a/ab/Jamies_4_hour_lamb_lamb_shanks.jpg?itok=_3CmMqyB",
"recipe_detail_750_550": "/sites/default/files/styles/recipe_detail_750_550/https/www.cookipedia.co.uk/wiki/images/a/ab/Jamies_4_hour_lamb_lamb_shanks.jpg?itok=tK9pG_LQ"
}
}Proposed resolution
Use absolute URLs
Remaining tasks
User interface changes
API changes
Data model changes
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 2911330-4.patch | 1.76 KB | dawehner |
| #4 | interdiff-2911330.txt | 1.66 KB | dawehner |
| #2 | 2911330-2.patch | 1.82 KB | dawehner |
Comments
Comment #2
dawehnerNote: This patch doesn't work yet ...
Comment #4
dawehnerThere we go ...
Comment #5
e0ipsoThis patch breaks BC. Let's make sure it's worth it.
Comment #7
dawehnerMaybe that's indeed something for version 2 of the module? By using
file_create_urlyou would theoretical enable CDN support for example.Comment #8
e0ipso@dawehner version 2 is released. Version 1 is now deprecated. Let's be agile!
Comment #9
dawehnerEmbrace the version control
Comment #10
e0ipso