Setting administer>settings>img_assist>Max inline image size allows entry of malformed value; seems to trigger the "Image code inserted but the image is not visible" bug
http://drupal.org/node/55882 (see posting # 12 for details)
Steps to reproduce:
- Enter "400" in the administer>settings>img_assist>Max inline image size field.
- Try to insert an image into a new node using img_assist tags.
- Image does not appear (no img tags are emitted.)
- All subsequent edits to nodes having previously inserted img_assist tags will also fail.
Also it appears that this value can be corrupted during an upgrade (some have reported that it happens after an upgrade to 4.7.0) but I've not confirmed that, and that's another bug report... from http://drupal.org/node/55882 # 13 :
There was a huge number filled in (like 204583) after updating to 4.7.0 and img_assist cvs and I thought it was the file size or something else...
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | img_assist-64259.patch | 7.17 KB | darren oh |
| #6 | img_assist_22.patch | 7.17 KB | sun |
| #5 | img_assist_21.patch | 6.71 KB | sun |
Comments
Comment #1
Folkert commentedAlso when the image is 300x300 and you insert the 'filter' into the textarea, the image does not display. After changing the width and heigth values, in the filter tag, to a value > then the actual size the image appears as aspected.
example
The image actual size is 300 x 300
the inserted filter tag shows
[img_assist|nid=31|title=testImage|desc=|link=none|align=left|width=302|height=300]result is a picture NOT showing.
Then i change the filter to bigger then the actual size like:
[img_assist|nid=31|title=testImage|desc=|link=none|align=left|width=310|height=310]and bingo there you go the picture shows and i'm happy.
Will look into the code hopefully this weekend.
Comment #2
Folkert commentedsorry using version 4.7.2 drupal with latest img_assist
Comment #3
Boinng commentedFolkert - that's a separate issue which is documented here - http://drupal.org/node/60771 - unfortunately the only patch so far hasn't solved it for me..
Comment #4
darren ohComment #5
sunThat's true and should be fixed with the attached patch.
While being there, I've additionally re-phrased some other settings. Besides the "priviledged users" thing, I already know translators will take me out for that. So we should ensure that those phrases are future-proof. I know my English is not always the best, so this should be reviewed by someone else.
Comment #6
sunNew patch, additionally fixing options for default size for inline images (empty preset sizes were displayed, such as 'original' by default) and format_plural placeholder (needs to be @count in D5).
Comment #7
darren ohFound two typos, but otherwise the patch was fine. Typos are fixed in this patch.
Comment #8
sunAwesome! Thanks for reviewing and updating the patch, Darren!
Comment #9
(not verified) commented