Closed (fixed)
Project:
Insert
Version:
6.x-1.0-beta1
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Oct 2009 at 23:17 UTC
Updated:
17 Nov 2009 at 16:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
quicksketchThanks Wim, I had this occurring before I release Insert module, but I thought I had it corrected. This is probably a configuration I didn't test. Let me guess you have:
- An imagefield on which Insert is enabled.
- The imagefield does not have an alt or title field enabled.
I think I only tested it on an imagefield that did have both alt and title.
Comment #2
quicksketchI think this patch should correct the problem.
Comment #3
quicksketchYep, I reproduced this problem by disabling alt/title fields. The above patch corrects the problem and I've committed it.
Comment #4
wim leersYour guesses were correct! And thanks for the fix :)
Comment #5
wim leersAnd I didn't notice this, but apparently the generated URLs are now absolute instead of relative? I don't mind. Just mentioning this changed behavior. This can also be seen in the screenshot I posted.
Feel free to mark as fixed again :)
Comment #6
quicksketchYes the URLs are now absolute per #521812: relative path or full path settings. Basically full URLs are much more useful than relative ones for RSS feeds and aggregated content, even if they're less compatible between dev/live. I'd recommend Pathologic if that's a problem, but absolute URLs I think give the better "out-of-box" experience.
Comment #7
wim leersOk, thanks for the follow-up!