Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

パスに関する不具合 #2

Merged
merged 2 commits into from

2 participants

@haru

redmine_attach_screenshotをこちらの環境にインストールして2点ほど上手く動かない点がありましたので修正しました。問題が無ければ取り込んでいただけないでしょうか?

  1. RedmineのURLをサブディレクトリに設定するとアイコンが表示されない。
    アイコンの場所指定が "/images/~" のように/で始まっているため、
    http://my.server/redmine のようにRedmineがサブディレクトリに配置されている場合にアイコンが表示されません。
    サブディレクトリに配置されてもアイコンを取得できるよう修正しました。

  2. routes.rbを変更していると画像を添付できない。
    attach_screenshot_controllerのURLの決定をwelcom_controllerのURLを基準に行っていますが、こちらの環境ではroutes.rbを変更してwelcom_controllerのURLが'/'でなくなってしまっているためこれだと上手く動きませんでした。
    #ちなみにroutes.rbを変更しているのはデフォルトページをwelcomページではなくメインプロジェクトの概要ページにしているため。
    URLの決定にwelcome_controllerを使用しないように修正してみました。

@mikoto20000 mikoto20000 merged commit d6de788 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 24, 2012
This page is out of date. Refresh to see the latest.
View
13 app/views/attachments/_form.rhtml
@@ -1,3 +1,6 @@
+<%
+baseurl = Redmine::Utils.relative_url_root
+-%>
<% content_for :header_tags do %>
<%= stylesheet_link_tag 'attach_screenshot', :plugin => 'redmine_attach_screenshot' %>
<%= javascript_include_tag 'attach_screenshot', :plugin => 'redmine_attach_screenshot' %>
@@ -42,14 +45,14 @@
<script type="text/javascript">
function showAttachScreen() {
document.getElementById('attach_applet').style.display = 'block';
- document.getElementById('attach_applet').innerHTML = '<applet codebase="<%= url_for(:only_path => false, :controller => 'welcome')%>plugin_assets/redmine_attach_screenshot/javascripts/" code="com.axmor.redmine.uploader.Uploader.class" archive="uploader_applet.jar" height="500" width="600" MAYSCRIPT>'+
+ document.getElementById('attach_applet').innerHTML = '<applet codebase="<%= url_for(:only_path => false, :controller => 'attach_screenshot')%>/../plugin_assets/redmine_attach_screenshot/javascripts/" code="com.axmor.redmine.uploader.Uploader.class" archive="uploader_applet.jar" height="500" width="600" MAYSCRIPT>'+
'<param name="label.image" value="<%= l(:label_image) %>">' +
'<param name="label.button.paste" value="<%= l(:label_button_paste) %>">' +
'<param name="label.button.attach" value="<%= l(:button_attach) %>">' +
'<param name="label.button.cancel" value="<%= l(:button_cancel) %>">' +
'<param name="error.close.msg" value="<%= l(:error_close_msg) %>">' +
'<param name="error.close.title" value="<%= l(:error_close_title) %>">' +
- '<param name="attach.url" value="<%= url_for(:only_path => false, :controller => 'welcome')%>attach_screenshot">' +
+ '<param name="attach.url" value="<%= url_for(:only_path => false, :controller => 'attach_screenshot')%>">' +
'<param name="error.attach.msg" value="<%= l(:error_attach_msg) %>">' +
'<param name="error.attach.title" value="<%= l(:error_attach_title) %>">' +
'<param name="rss.key" value="<%= User.current.rss_key %>">' +
@@ -66,16 +69,16 @@
s2.className = "st2";
var im = document.createElement("img");
- im.src = "<%= url_for(:only_path => false, :controller => 'welcome')%>attach_screenshot?id=" + fileId;
+ im.src = "<%= url_for(:only_path => false, :controller => 'attach_screenshot')%>?id=" + fileId;
var b = document.createElement("input");
b.type = "image";
- b.src = "/images/delete.png"
+ b.src = "<%= baseurl -%>/images/delete.png"
b.setAttribute("onClick", "deleteAttachScreen('" + fileId + "'); return false;");
var i = document.createElement("input");
i.type = "image";
- i.src = "/images/lightning.png";
+ i.src = "<%= baseurl -%>/images/lightning.png";
i.setAttribute("onClick", "InsertAttachScreen('" + fileId + "'); return false;");
var d = document.createElement("input");
View
2  assets/stylesheets/attach_screenshot.css
@@ -40,7 +40,7 @@ span.screen_thumb .st2 {
background-color: #F7F7F7;
background-position: right;
background-repeat: no-repeat;
- background-image: url(/plugin_assets/redmine_attach_screenshot/images/capture-24x24.png);
+ background-image: url(../images/capture-24x24.png);
}
.screenshot_button span {
Something went wrong with that request. Please try again.