View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0021133 | mantisbt | rss | public | 2016-06-17 03:20 | 2020-03-15 15:23 |
Reporter | atrol | Assigned To | dregad | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.2.9 | ||||
Target Version | 2.24.0 | Fixed in Version | 2.24.0 | ||
Summary | 0021133: Access of non existent image in RSS feeds | ||||
Description | File mantis_logo_button.gif has been removed in commit MantisBT master e47a2dcb when our new logo has been introduced. issues_rss.php news_rss.php | ||||
Tags | No tags attached. | ||||
Amazing that the bug has been present for so long (2012), and considering how simple the fix, that nobody bothered to patch it since this report was opened in 2016... |
|
@dregad I don't remember, maybe because I thought that we need a new image, as the old image was just 88 x 35 pixel. |
|
Hmm, good point I did not verify that there was a restriction on image size. The RSS 2.0 specification states
Our logo is 232x80, so it actually exceeds the maximum width. I'm not sure what consequence this would have on RSS clients (error, resized image, no display...). I hardly use RSS anymore these days, and I don't know of a local client supporting channel images that I can use for testing (it needs to be local so I can test against my dev box's localhost setup). Maybe @LewisR or @Camille Desmots can provide feedback on this. In any case, IMO the proposed patch is no worse than the current situation of referencing the wrong logo ;-) |
|
Hmmm... Well, my reading is slightly different. From the text just above the maximum values stated: Optional elements include <width> and <height>, numbers, indicating the width and height of the image in pixels. <description> contains text that is included in the TITLE attribute of the link formed around the image in the HTML rendering. which indicates to me that those values are not image size limits but rather value limits when values have been entered using the optional elements. My feed reader is SeaMonkey, and I don't see these images at all (and I was fairly certain that my own blog includes them - I'll need to look at the xml). Sorry I can't be of more help at this particular moment. |
|
OK, thanks for the feedback anyway @LewisR. |
|
MantisBT: master e47a2dcb 2012-05-06 07:30 Details Diff |
New logo, button and favicon. 1. Logo - for now login page and normal pages use the same logo size. In future versions, we may include a slightly larger one for the login page. Didn't want to include this in a minor update, since this is a configurable image name, and hence adding a new logo, may end up with having a mix of two logos. 2. The button is now a scaled version of the logo. For this reason, it is removed from the footer of the login page, but kept on the others. 3. Added the new favicon. |
Affected Issues 0014119, 0021133 |
|
add - images/mantis_logo_232x80.png | Diff File | ||
rm - images/mantis_logo.gif | Diff | ||
rm - images/mantis_logo_button.gif | Diff | ||
mod - config_defaults_inc.php | Diff File | ||
mod - core/html_api.php | Diff File | ||
mod - images/favicon.ico | Diff File | ||
MantisBT: master a91e7e13 2020-01-28 01:55 Details Diff |
Use $g_logo_image for RSS feeds The code was using an hardcoded image name (mantis_logo_button.gif), obsolete since the introduction of the new MantisBT logo, back in 2012 / 1.2.9 (!) Fixes 0021133 |
Affected Issues 0021133 |
|
mod - issues_rss.php | Diff File | ||
mod - news_rss.php | Diff File |