Geeklog-jp / geeklog-jp

This is Geeklog 2.1.0 JP 1.1. Geeklog 2.1.1 JP 1.0: see https://github.com/Geeklog-jp/geeklog-japan
http://www.geeklog.jp
0 stars 1 forks source link

インストーラーから,precheck.php へのリンクを追加 #104

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
問題を再現するための操作手順を記入して下さい。
1.インストーラーをたちあげる。
2.
3.

正常に動作する場合に、期待される出力(表示)はなん
ですか?

precheck.php へのリンクがある。

実際にはどのように出力(表示)されますか?

precheck.php へのリンクがない。

Geeklogのリリースまたはスナップショットのバージョン,
Subversionから取り出した場合は元のディレクトリと
リビジョンは何ですか?

Geeklog1.6.0sr2-jp

ラベルとテキストを使用して、できるだけ詳しく情報を
提供して下さい。

インストールで困ったら、こちらのサイトへ 
の左隣あたりに,
日本語環境のインストール前チェック
として precheck.phpへリンクしてほしい。

Original issue reported on code.google.com by ivysoho on 21 Oct 2009 at 12:36

GoogleCodeExporter commented 9 years ago
日本語言語ファイルに,以下を追加。日本独自仕様なので��
�言語ファイルの変数をどうするか
検討要

    106 => '日本語環境のインストール前チェック',
    107 => 'precheck.php'

        <div class="header-navigation-line">
            <a href="' . $LANG_INSTALL[107] . '" class="header-navigation">' .
$LANG_INSTALL[106] . '</a>   
            <a href="' . $LANG_INSTALL[87] . '" class="header-navigation">' .
$LANG_INSTALL[1] . '</a>   
        </div>

とすることで,インストーラを実行して日本語モードにす��
�と,右上に,インストールで困っ
たら、こちらのサイトへ だけでなく,日本語環境のイン��
�トール前チェック へのリンクが
表示されます。

日本語環境のインストール前チェック   
インストールで困ったら、こちらのサイトへ    

となります。

mystral-kkさん,いかがでしょうか。

対応よろしくお願いします。

Original comment by ivysoho on 21 Oct 2009 at 3:32

GoogleCodeExporter commented 9 years ago
上記修正ソースファイルは,index.php 920行目あたりです。

Original comment by ivysoho on 21 Oct 2009 at 3:34

GoogleCodeExporter commented 9 years ago
r2105で対応しました。言語データを既存の$LANG_INSTALLに追加��
�ると、本家の以降の更新と
バッティングする可能性があるため、$LANG_INSTALL_JPという変�
��を言語ファイルの末尾に追
加して使用しています。

Original comment by mystralkk@gmail.com on 21 Oct 2009 at 11:40

GoogleCodeExporter commented 9 years ago
mystralkkさん,どうもありがとうございました!

1.
$LANG_INSTALL_JPのglobal宣言が抜けていましたのでよろしくお願�
��します。

    global $_CONF, $_TABLES, $LANG_INSTALL, $LANG_CHARSET, $_DB, $_DB_dbms,
$_DB_table_prefix, $_URL, $gl_path, $html_path, $dbconfig_path, 
$siteconfig_path,
$display, $language, $form_label_dir, $use_innodb, $LANG_INSTALL_JP;

2. 表示順番について

日本語環境のインストール前チェック 
は,インストール初期だけ表示されます。

インストールで困ったら、こちらのサイトへ は,他のイ��
�ストール関係のページすべてで,
右上に表示されるものです。

ですので,
            <a href="' . 'precheck.php' . '" class="header-navigation">' .
$LANG_INSTALL_JP[1] . '</a>   
            <a href="' . $LANG_INSTALL[87] . '" class="header-navigation">' .
$LANG_INSTALL[1] . '</a>   

のように順番を変更したほうが良いと思いますがいかがで��
�ょうか。

Original comment by ivysoho on 21 Oct 2009 at 11:59

GoogleCodeExporter commented 9 years ago
> 
1.$LANG_INSTALL_JPのglobal宣言が抜けていましたのでよろしく�
��願いします。

$LANG_INSTALL_JPはスクリプトのトップレベルでしか使用してい�
��いので、global宣言は不要
です。

> 2. 表示順番について
> 日本語環境のインストール前チェック 
は,インストール初期だけ表示されます。
> 
インストールで困ったら、こちらのサイトへ は,他のイ��
�ストール関係のページすべてで,
> 右上に表示されるものです。

> ですので,
>            <a href="' . 'precheck.php' . '" class="header-navigation">' .
> $LANG_INSTALL_JP[1] . '</a>   
>            <a href="' . $LANG_INSTALL[87] . '" class="header-navigation">' .
> $LANG_INSTALL[1] . '</a>   
> 
のように順番を変更したほうが良いと思いますがいかがで��
�ょうか。

Ivyさんの好きなようにしてください。

Original comment by mystralkk@gmail.com on 22 Oct 2009 at 8:58

GoogleCodeExporter commented 9 years ago
global宣言をしない場合ままだと,言語を日本語に切り替え��
�ときは出ませんが,最初の英語
の表示や他の言語に変更したとき,
Notice: Undefined variable: LANG_INSTALL_JP in 
/path_to_html/admin/install/index.php
on line 920
と表示されてしまいますよ。

Original comment by ivysoho on 22 Oct 2009 at 9:17

GoogleCodeExporter commented 9 years ago
日本語以外の言語ではそもそも$LANG_INSTALL_JPが存在しない点�
��問題なので、r2109でしかる
べき対処を行いました。

Original comment by mystralkk@gmail.com on 22 Oct 2009 at 9:56

GoogleCodeExporter commented 9 years ago
ありがとうございます。
r2107, r2109で,動作確認完了しました。

StatusをFixedに変更します。

Original comment by ivysoho on 22 Oct 2009 at 10:03