http://redmine.openpne.jp/
http://redmine.openpne.jp/favicon.ico
2011-07-21T11:12:56Z
OpenPNE Issue Tracking System
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=9706
2011-07-21T11:12:56Z
Mutsumi Imamura
imamura@openpne.jp
<ul><li><strong>対象バージョン</strong> を <i>OpenPNE 3.7.0</i> にセット</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12481
2011-12-16T09:08:13Z
Yuma Sakata
sakata@tejimaya.com
<ul><li><strong>3.6 で発生するか</strong> を <i>Yes (はい)</i> にセット</li><li><strong>3.4 で発生するか</strong> を <i>Yes (はい)</i> にセット</li></ul><a name="再現確認"></a>
<h3 >再現確認<a href="#再現確認" class="wiki-anchor">¶</a></h3>
<p>以下バージョンで再現確認できました。</p>
<ul>
<li>3.6.1</li>
<li>3.4.18</li>
</ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12606
2012-01-16T10:19:46Z
Yuma Sakata
sakata@tejimaya.com
<ul><li><strong>優先度</strong> を <i>Normal(通常)</i> から <i>High(高め)</i> に変更</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12696
2012-01-31T16:58:43Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>New(新規)</i> から <i>Accepted(着手)</i> に変更</li><li><strong>担当者</strong> を <i>Yuya Watanabe</i> にセット</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12697
2012-01-31T17:04:34Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>説明</strong> を更新 (<a title="差分を表示" href="/journals/12697/diff?detail_id=16472">diff</a>)</li></ul><p>修正内容の修正パッチリンク先が存在しなかったため別の方法で修正.<br />修正内容の二番目の実装の必要性がわからなかったため一番目の修正内容の実装のみを行った.</p>
<p>以下は以前チケットに書かれていた内容.<br /><pre>
h3. Way to fix (修正内容)
* parse_url() の戻り値配列に含まれる port を保持。(lib/config/opApplicationConfiguration.class.php)
* sfPatternRoutingを継承したクラス opPatternRouting で修正。symfony側(sfRouting)には手をつけない。
修正パッチ: https://github.com/tozuka/OpenPNE3/commit/1da0e7306a7815a8502f7524a6b26f1436096bd7
h3. 備考
config/OpenPNE.yml.sample のように<pre>
base_url: "http://example.com"
</pre>が指定されていると、実際のリクエストからURLをポート番号込みで読み取って絶対URLを生成してくれる実装になっているが、op_base_url までは書き換えてくれないのでメールのフッタには http://example.com と表示される。
</pre></p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12699
2012-01-31T17:11:36Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Accepted(着手)</i> から <i>Pending Review(レビュー待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>0</i> から <i>50</i> に変更</li></ul><p>更新履歴 <a class="changeset" title="(fixes #2008) add port to generated url" href="http://redmine.openpne.jp/projects/op3/repository/revisions/bdca873709d66f3fb999b3fd1efd824a4e1b8ecc">bdca873709d66f3fb999b3fd1efd824a4e1b8ecc</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12768
2012-02-06T08:41:30Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Pending Review(レビュー待ち)</i> から <i>Accepted(着手)</i> に変更</li><li><strong>進捗率</strong> を <i>50</i> から <i>0</i> に変更</li></ul><p>sslを用いる場合に正常に動作しない可能性があるため,チケットのステータスを再度「Accepted」に戻します.</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12782
2012-02-06T12:01:58Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>説明</strong> を更新 (<a title="差分を表示" href="/journals/12782/diff?detail_id=16594">diff</a>)</li></ul><a name="修正方法"></a>
<h3 >修正方法<a href="#修正方法" class="wiki-anchor">¶</a></h3>
<p>note-5 で書き換える前の方針で修正を行った.<br />つまり,ベースにする URL は実際に生成するまで不明なため, opPatternRouting の generate() が呼び出されるときに決定するように修正を行った.</p>
<pre>
diff --git a/lib/config/opApplicationConfiguration.class.php b/lib/config/opApplicationConfiguration.class.php
index cb69ad2..e6158f3 100644
--- a/lib/config/opApplicationConfiguration.class.php
+++ b/lib/config/opApplicationConfiguration.class.php
@@ -582,35 +582,9 @@ abstract class opApplicationConfiguration extends sfApplicationConfiguration
sfConfig::set('sf_app', $application);
$configuration->setAppDir(sfConfig::get('sf_apps_dir').DIRECTORY_SEPARATOR.$application);
- $settings = sfDefineEnvironmentConfigHandler::getConfiguration($configuration->getConfigPaths('config/settings.yml'));
- $isNoScriptName = !empty($settings['.settings']['no_script_name']);
-
$options = $context->getRouting()->getOptions();
- $url = sfConfig::get('op_base_url');
- if ('http://example.com' !== $url)
- {
- $parts = parse_url($url);
-
- $parts['path'] = isset($parts['path']) ? $parts['path'] : '';
- $options['context']['prefix'] =
- $this->getAppScriptName($application, sfConfig::get('sf_environment'), $parts['path'], $isNoScriptName);
-- if (isset($parts['host']))
- {
- $options['context']['host'] = $parts['host'];
- if (isset($parts['port']))
- {
- $options['context']['host'] .= ':'.$parts['port'];
- }
- }
- }
- else
- {
- $path = preg_replace('#/[^/]+\.php$#', '', $options['context']['prefix']);
- $options['context']['prefix'] = $this->getAppScriptName($application, sfConfig::get('sf_environment'), $path, $isNoScriptName);
- }
- $routing = new sfPatternRouting($context->getEventDispatcher(), null, $options);
+ $routing = new opPatternRouting($context->getEventDispatcher(), null, $options);
$routing->setRoutes($config->evaluate($configuration->getConfigPaths('config/routing.yml')));
$context->getEventDispatcher()->notify(new sfEvent($routing, 'routing.load_configuration'));
diff --git a/lib/routing/opPatternRouting.class.php b/lib/routing/opPatternRouting.class.php
new file mode 100644
index 0000000..9f8ac7e
--- /dev/null
+++ b/lib/routing/opPatternRouting.class.php
@@ -0,0 +1,97 @@
+<?php
+
+/**
+ * This file is part of the OpenPNE package.
+ * (c) OpenPNE Project (http://www.openpne.jp/)
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file and the NOTICE file that were distributed with this source code.
+ */
+class opPatternRouting extends sfPatternRouting
+{
+
+ public function generate($name, $params = array(), $absolute = false)
+ {
+ $app = sfConfig::get('sf_app');
+ $sslAppRequiredList = sfConfig::get('op_ssl_required_applications', array());
+ $sslActionRequiredList = sfConfig::get('op_ssl_required_actions', array($app => array()));
+
+ if (sfConfig::get('op_use_ssl', false)
+ && in_array($app, $sslAppRequiredList)
+ && isset($params['module']) && isset($params['action'])
+ && in_array($params['module'].'/'.$params['action'], $sslActionRequiredList[$app])
+ )
+ {
+ $this->options['context']['is_secure'] = true;
+ $sslBaseUrls = sfConfig::get('op_ssl_base_url');
+ $url = $sslBaseUrls[$app];
+ $isDefault = 'https://example.com' === $url;
+ }
+ else
+ {
+ $this->options['context']['is_secure'] = false;
+ $url = sfConfig::get('op_base_url');
+ $isDefault = 'http://example.com' === $url;
+ }
+
+ $parts = parse_url($url);
+
+ $configuration = sfContext::getInstance()->getConfiguration();
+ $settings = sfDefineEnvironmentConfigHandler::getConfiguration($configuration->getConfigPaths('config/settings.yml'));
+ $isNoScriptName = !empty($settings['.settings']['no_script_name']);
+
+ if (!$isDefault)
+ {
+ $parts['path'] = isset($parts['path']) ? $parts['path'] : '';
+ $this->options['context']['prefix'] =
+ $this->getAppScriptName($app, sfConfig::get('sf_environment'), $parts['path'], $isNoScriptName);
+
+ if (isset($parts['host']))
+ {
+ $this->options['context']['host'] = $parts['host'];
+ if (isset($parts['port']))
+ {
+ $this->options['context']['host'] .= ':'.$parts['port'];
+ }
+ }
+ }
+ else
+ {
+ $path = preg_replace('#/[^/]+\.php$#', '', $this->options['context']['prefix']);
+ $this->options['context']['prefix'] = $this->getAppScriptName($app, sfConfig::get('sf_environment'), $path, $isNoScriptName);
+ }
+
+ return parent::generate($name, $params, $absolute);
+ }
+
+ // equals opApplicationConfiguration#getAppScriptName
+ private function getAppScriptName($application, $env, $prefix, $isNoScriptName = false)
+ {
+ if ($isNoScriptName)
+ {
+ return $prefix;
+ }
+
+ if ('/' === $prefix)
+ {
+ $prefix = '';
+ }
+
+ $name = $prefix.'/'.$application;
+ if ($env !== 'prod')
+ {
+ $name .= '_'.$env;
+ }
+ $name .= '.php';
+
+ return $name;
+ }
+
+}
</pre>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12783
2012-02-06T12:04:25Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>説明</strong> を更新 (<a title="差分を表示" href="/journals/12783/diff?detail_id=16595">diff</a>)</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12785
2012-02-06T12:10:04Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Accepted(着手)</i> から <i>Pending Review(レビュー待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>0</i> から <i>50</i> に変更</li></ul><p>更新履歴 <a class="changeset" title="(fixes #2008) fixed to generate url considering SSL and the port" href="http://redmine.openpne.jp/projects/op3/repository/revisions/9be543bd53d57c94a545968057441e5ff13c6d43">9be543bd53d57c94a545968057441e5ff13c6d43</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12787
2012-02-06T12:20:10Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>説明</strong> を更新 (<a title="差分を表示" href="/journals/12787/diff?detail_id=16602">diff</a>)</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12814
2012-02-07T11:36:02Z
Yuya Watanabe
watanabe@openpne.jp
<ul></ul><p>更新履歴 <a class="changeset" title="Revert "(fixes #2008) fixed to generate url considering SSL and the port" This reverts commit 9b..." href="http://redmine.openpne.jp/projects/op3/repository/revisions/4baf79713251326b65fc452ce43002c7153186d4">4baf79713251326b65fc452ce43002c7153186d4</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12815
2012-02-07T11:36:03Z
Yuya Watanabe
watanabe@openpne.jp
<ul></ul><p>更新履歴 <a class="changeset" title="(fixes #2008) fixed to generate url considering SSL and the port" href="http://redmine.openpne.jp/projects/op3/repository/revisions/d65df79b24955e1302c89b65d982a44758c957a7">d65df79b24955e1302c89b65d982a44758c957a7</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12816
2012-02-07T11:40:29Z
Yuya Watanabe
watanabe@openpne.jp
<ul></ul><p>note-8 の内容は revert し, 現在のコンテキストが is_secure ならば ssl_base_url を見るように変更.</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12817
2012-02-07T11:49:08Z
Kousuke Ebihara
ebihara@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>Pending Review(レビュー待ち)</i> から <i>Pending Testing(テスト待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>50</i> から <i>70</i> に変更</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12883
2012-02-08T10:42:35Z
Minoru Takai
main.coeurl@gmail.com
<ul><li><strong>ステータス</strong> を <i>Pending Testing(テスト待ち)</i> から <i>Rejected(差し戻し)</i> に変更</li><li><strong>進捗率</strong> を <i>70</i> から <i>50</i> に変更</li></ul><a name="ロジックが間違っていないか"></a>
<h3 >ロジックが間違っていないか<a href="#ロジックが間違っていないか" class="wiki-anchor">¶</a></h3>
<p>note-15 でレビューが通っていますが、一点確認させてください。</p>
<pre>
@@ -586,8 +586,19 @@ abstract class opApplicationConfiguration extends sfApplicationConfiguration
$isNoScriptName = !empty($settings['.settings']['no_script_name']);
$options = $context->getRouting()->getOptions();
- $url = sfConfig::get('op_base_url');
- if ('http://example.com' !== $url)
+ if ($options['context']['is_secure'])
+ {
+ $sslBaseUrls = sfConfig::get('op_ssl_base_url');
+ $url = $sslBaseUrls[$application];
+ $isDefault = 'https://example.com' === $url;
+ }
+ else
+ {
+ $url = sfConfig::get('op_base_url');
+ $isDefault = 'http://example.com' === $url;
+ }
+
+ if ($isDefault)
{
$parts = parse_url($url);
</pre>
<p>もともとは</p>
<pre>
$url = sfConfig::get('op_base_url');
if ('http://example.com' !== $url)
{
$parts = parse_url($url);
</pre>
<p>と、 $url を取得し、それが example.com ではない場合に if 文に入っていますが、修正後は、</p>
<pre>
if ($options['context']['is_secure'])
{
$sslBaseUrls = sfConfig::get('op_ssl_base_url');
$url = $sslBaseUrls[$application];
$isDefault = 'https://example.com' === $url;
}
else
{
$url = sfConfig::get('op_base_url');
$isDefault = 'http://example.com' === $url;
}
if ($isDefault)
{
$parts = parse_url($url);
</pre>
<p>$url を取得し、それが example.com である場合( $isDefault である場合)に if 文に入っています。これは意図した修正でしょうか。コミットログや note-14 からは、この条件式を反転させたことが妥当なのか評価できませんでした。</p>
<a name="isDefault-部分の保守性と可読性"></a>
<h3 >$isDefault 部分の保守性と可読性<a href="#isDefault-部分の保守性と可読性" class="wiki-anchor">¶</a></h3>
<p>補足程度ですが、 $isDefault への代入を各ブロックに記述するよりも外に出した方が保守性が高いように思います。更に、 $isDefault をこの if 文一箇所でしか使っていない(他所で使う予定がない)のであればこの変数は不要かもしれません。</p>
<pre>
if ($options['context']['is_secure'])
{
$sslBaseUrls = sfConfig::get('op_ssl_base_url');
$url = $sslBaseUrls[$application];
$urlScheme = 'https://';
}
else
{
$url = sfConfig::get('op_base_url');
$urlScheme = 'http://';
}
if ($urlScheme.'example.com' === $url) // ここは等号比較ではなく、否定等号比較か
{
$parts = parse_url($url);
</pre>
<ul>
<li><a class="external" href="http://tools.ietf.org/html/rfc3986#page-16">http://tools.ietf.org/html/rfc3986#page-16</a> (下記)あたりを見ても 'http://' までの文字列を表す適切な語が分かりませんでした。下記でいう scheme はより具体的には schemeName と表現できそうです。 schemePart はまた別の定義があるようなので scheme "://" を表すには適切ではなさそうです。ということで上記では $urlScheme という変数名を使ってみました。<br /><pre>
URI = scheme ":" hier-part [ "?" query ] [ "#" fragment ]
hier-part = "//" authority path-abempty
/ path-absolute
/ path-rootless
/ path-empty
</pre></li>
</ul>
<p>さらに、ここまで書いて思いましたが、 URL が example.com かどうかを調べるだけであれば、 (http|https):// を用意して URL の完全一致比較をおこなうよりも、正規表現で書いたほうが可読性が高いかもしれません。</p>
<pre>
if ($options['context']['is_secure'])
{
$sslBaseUrls = sfConfig::get('op_ssl_base_url');
$url = $sslBaseUrls[$application];
}
else
{
$url = sfConfig::get('op_base_url');
}
if (preg_match('#^https?://example\.com$#', $url)) // ここは preg_match() ではなく、その否定 !preg_match() か
{
$parts = parse_url($url);
</pre>
<ul>
<li>^$ ではなく \A\z を使うべきだという話がありますが、ここではその話は考慮していません。</li>
</ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12886
2012-02-08T12:53:41Z
Yuya Watanabe
watanabe@openpne.jp
<ul></ul><a name="問題"></a>
<h3 >問題<a href="#問題" class="wiki-anchor">¶</a></h3>
<p>note-8 に記述されているコードを revert し,再度修正を行う際に lib/config/opApplicationConfiguration.class.php の 601 行目の $isDefault の部分を考慮することが抜けていました.その部分については以下のように修正します.</p>
<p>$isDefault の保守性についてですが,今後この設定値の部分が変化する可能性が多分にある場合はこの部分について何らかの手を打つべきではあると思います.<br />正規表現や部分的な共通化などの処理を行うことによって可読性や可用性が向上することが見込めますが,そのように幅をもたせるような記述をした場合に不具合の原因となる可能性が高くなってしまいます.<br />(例えば note-16 のような正規表現では base_url が <a class="external" href="https://example.com">https://example.com</a> と書き換えられた場合でもデフォルト値として認識してしまいます.これが問題かどうかは今回とは別の議論となりますが.)<br />今回の場合では この部分は固定値の完全一致によってデフォルト値かどうかを判定することができ,今後変化することも現状考えられないと思います.<br />不具合の可能性を高めるようなリスクを負うほどの利点は得られないと考えて, $isDefault を正規表現や部分的な共通化などの対策については避ける方向で行きます.</p>
<a name="修正内容"></a>
<h3 >修正内容<a href="#修正内容" class="wiki-anchor">¶</a></h3>
<pre>
diff --git a/lib/config/opApplicationConfiguration.class.php b/lib/config/opApplicationConfiguration.class.php
index 5e32e01..09f653b 100644
--- a/lib/config/opApplicationConfiguration.class.php
+++ b/lib/config/opApplicationConfiguration.class.php
@@ -598,7 +598,7 @@ abstract class opApplicationConfiguration extends sfApplicationConfiguration
$isDefault = 'http://example.com' === $url;
}
- if ($isDefault)
+ if (!$isDefault)
{
$parts = parse_url($url);
</pre>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12890
2012-02-09T04:16:14Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Rejected(差し戻し)</i> から <i>Accepted(着手)</i> に変更</li><li><strong>進捗率</strong> を <i>50</i> から <i>0</i> に変更</li></ul>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12891
2012-02-09T04:40:58Z
Yuya Watanabe
watanabe@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Accepted(着手)</i> から <i>Pending Review(レビュー待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>0</i> から <i>50</i> に変更</li></ul><p>更新履歴 <a class="changeset" title="(fixes #2008) fixed to flip default flag to evaluate correct boolean" href="http://redmine.openpne.jp/projects/op3/repository/revisions/2ed9c936f62482e4c3d5ea91954b01943873a47c">2ed9c936f62482e4c3d5ea91954b01943873a47c</a> で適用されました。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=12906
2012-02-09T07:46:55Z
Kousuke Ebihara
ebihara@tejimaya.com
<ul><li><strong>ステータス</strong> を <i>Pending Review(レビュー待ち)</i> から <i>Pending Testing(テスト待ち)</i> に変更</li><li><strong>進捗率</strong> を <i>50</i> から <i>70</i> に変更</li></ul><p>確認しました。</p>
<p>``$isDefault`` の共通化の必要は現状ないと思いますし、可読性の面でも充分です(正規表現等でのマッチングに比べればこちらのアプローチのほうが優れています)。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=13385
2012-03-08T06:09:36Z
Shouta Kashiwagi
kashiwagi@openpne.jp
<ul><li><strong>ステータス</strong> を <i>Pending Testing(テスト待ち)</i> から <i>Fixed(完了)</i> に変更</li><li><strong>進捗率</strong> を <i>70</i> から <i>100</i> に変更</li></ul><p>テストOKです。</p>
OpenPNE 3 - Bug(バグ) #2008: config/OpenPNE.yml で80番以外のポート番号を含むURLをbase_urlに指定した場合、通知メール等に含まれるURLにポート番号が反映されない
http://redmine.openpne.jp/issues/2008?journal_id=19514
2015-10-08T09:15:46Z
kaoru n
<ul><li><strong>3.8 で発生するか</strong> を <i>Unknown (未調査)</i> にセット</li></ul>