#556 closed defect (fixed)
Screenshots don't show up
Reported by: | Scott Grayban | Owned by: | Radek Bartoň |
---|---|---|---|
Priority: | highest | Component: | ScreenshotsPlugin |
Severity: | blocker | Keywords: | |
Cc: | Trac Release: | 0.10 |
Description
Maybe I'm just stupid but I don't know where these screenshots are suppose to be viewed at.
I click on the nav menu for it and I only get the description for it and the source says no image.
<div id="content" class="screenshots"> <div class="title"> <h1>Screenshots</h1> </div> <div class="header"> <b>Component:</b> core (Non gui core bugs.) <b>Version:</b> 1.0 </div> <div class="images"> <div class="previous"> <a class="noimage" href="#"></a> <a class="noimage" href="#"></a> </div>
Attachments (0)
Change History (13)
comment:1 Changed 18 years ago by
comment:2 Changed 18 years ago by
Ahhhh I someting funky when I view the css for it
Traceback (most recent call last): File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 335, in dispatch_request dispatcher.dispatch(req) File "/usr/lib/python2.4/site-packages/trac/web/main.py", line 220, in dispatch resp = chosen_handler.process_request(req) File "/usr/lib/python2.4/site-packages/trac/web/chrome.py", line 225, in process_request for dir in [os.path.normpath(dir[1]) for dir File "/usr/lib/python2.4/site-packages/HackInstall-0.5-py2.4.egg/hackinstall/web_ui.py", line 164, in get_htdocs_dirs File "/usr/lib/python2.4/site-packages/setuptools-0.6c1-py2.4.egg/pkg_resources.py", line 799, in resource_filename return get_provider(package_or_requirement).get_resource_filename( File "/usr/lib/python2.4/site-packages/setuptools-0.6c1-py2.4.egg/pkg_resources.py", line 1229, in get_resource_filename return self._extract_resource(manager, zip_path) File "/usr/lib/python2.4/site-packages/setuptools-0.6c1-py2.4.egg/pkg_resources.py", line 1240, in _extract_resource zip_stat = self.zipinfo[zip_path] KeyError: 'hackinstall/htdocs'
comment:3 Changed 18 years ago by
Nevermind -- I still had that broken HackInstallPlugin still.
But still have issues with seeing the screenshots still.
Is path = <path_to_screenshots> full path or relative to the project? EG; site/screenshots/
Because the link to the screen shot is giving me http://dev-1.borgforge.net/castpodder/screenshots/castpodder_screenshot_large.png when it should be http://dev-1.borgforge.net/castpodder/chrome/site/screenshots/castpodder_screenshot_large.png
comment:4 Changed 18 years ago by
Status: | new → assigned |
---|
Hello,
path variable is absolute so you should set it as ie. /var/lib/trac/project_1/screenshots . If you set path as this example and your problems will be otlasting, please remove HackInstallPlugin or solve problems with it and post me log output here. Note that ImageMagick tools properly installed is needed for running this plugin.
comment:5 Changed 18 years ago by
Still not working and I removed the hackinstall already.
I think you need to set the path relative to the project. EG; site/screenshots where site would always be /var/trac/project/htdocs
Because the htdocs is the public web folder and anything outside of that will not work as a link.
comment:6 Changed 18 years ago by
whoops sorry -- I forgot to login.
This is what I have in my trac.ini
[screenshots] path = /var/trac/project/castpodder/htdocs/screenshots title = Screenshots
The link for the image is wrong no matter what I set it to.
You use a static link which is bad => http://dev-1.borgforge.net/castpodder/screenshots/castpodder_screenshot_large.png
When it SHOULD use => http://dev-1.borgforge.net/castpodder/chrome/site/screenshots/castpodder_screenshot_large.png
See the problem?
Your path is for saving the screenshots and not for the links/thumbs.
comment:7 Changed 18 years ago by
With URL http://dev-1.borgforge.net/castpodder/screenshots/castpodder_screenshot_large.png is nothing wrong. This URL is inside the plugin interpreted to filename and then appended to the end of path config variable and returned to the request object. So there is problem with this appending and returning. The reason why URL http://dev-1.borgforge.net/castpodder/chrome/site/screenshots/castpodder_screenshot_large.png is working is coincidence that you have stored you screenshots inside htdocs dir, but image is returned to the request by core trac modules. Of course you have to set you path variable to directory which is to the web server accesible (ie. has apache ownwer), but it can be everywhere on the system. I'll take a close look on the code to consider what could be wrong but debug log output should tell me more. Unfortunately I'm afraid that there is no enough relevant debug prints in the code so I'll make a new changeset which you could try evoke error with.
comment:8 Changed 18 years ago by
Your thinking apache -- you can't -- I and alot of others use the standalone server. It's faster then apache.
You need to make sure the thumbs and link for the pictures are set to use the template paths here.
Standalone server can't use rewrite rules.
The graphviz plugin is a good example to follow. He uses template paths to render and show the graphic. And a separate path for the URL.
[graphviz] cache_dir = /usr/share/trac/htdocs/graphviz prefix_url = http://dev-1.borgforge.net/castpodder/chrome/common/graphviz
comment:9 Changed 18 years ago by
By standalone server you mean tracd? Because I'm testing with tracd and there is no problem with it. I tried plugin with apache now and there problem is. Image is returned by req.send_file(path, type) so it should work everywhere.
comment:10 Changed 18 years ago by
I've commited new changeset 1078 which fix mistake in mime type returning. It should help so please try it.
comment:11 Changed 18 years ago by
It works now :)
Now all you need to do is add config options to set the default component and version to use.
comment:12 Changed 18 years ago by
Resolution: | → fixed |
---|---|
Severity: | major → blocker |
Status: | assigned → closed |
Oh and maybe a modify feature as well.
I do like this plugin since I can make screen shots depending on the version if any changes where changed.
Another cool feature would be the use of the tags plugin so you can also tag the screenshots.
But I will close this ticket and post a new one for the features.
comment:13 Changed 18 years ago by
I was considering make option to assing one screenhsot to more component or versions too.
The page text has