Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configmodify #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Configmodify #3

wants to merge 3 commits into from

Conversation

mikau
Copy link

@mikau mikau commented Oct 4, 2013

以下の修正をしてみました。

目的
 RSS表示、メニュー表示を選択できるようにする
 コンテンツ表示画面に表題を表示できるように
 カテゴリー名TOP、トップ表示をモジュール名に置き換えられるように
 デフォルトはい、で いいえ にすることの多いものを デフォルト いいえ に

修正履歴
xoops_version.php
ファイルの文字コードを utf-8にした
$modversion['author'] の値に全角文字があったものを半角に置き換えた

configを追加
 表題を表示する(デフォルト はい)
 モジュールトップページの時パンくずを隠す(デフォルト はい)
 TOPカテゴリーはモジュール名を表示する(デフォルト はい)
 RSSリンクを表示する(デフォルト いいえ)
 MENUリンクを表示する(デフォルト いいえ)

configのデフォルトを修正
 デフォルトを いいえ に
  ページナビゲーションを表示する
  印刷画面へのリンクを表示する
  友達に紹介するリンクを表示する
  tellafriendモジュールを利用する
  投票機能を利用する

mikau added 3 commits October 4, 2013 16:53
ファイルの文字コードを utf-8にした
$modversion['author'] の値に全角文字があったものを半角に置き換えた

configを追加
 表題を表示する(デフォルト はい)
 モジュールトップページの時パンくずを隠す(デフォルト はい)
 TOPカテゴリーはモジュール名を表示する(デフォルト はい)
 RSSリンクを表示する(デフォルト いいえ)
 MENUリンクを表示する(デフォルト いいえ)

configのデフォルトを修正
 デフォルトを いいえ に
  ページナビゲーションを表示する
  印刷画面へのリンクを表示する
  友達に紹介するリンクを表示する
  tellafriendモジュールを利用する
 デフォルトを いいえ に
  投票機能を利用する
@mikau
Copy link
Author

mikau commented Oct 5, 2013

languageに追加しました。別のブランチにした方がよかったですかね。

@nao-pon
Copy link
Member

nao-pon commented Oct 5, 2013

ありがとうございます。
マージするまでは同じブランチでOKです。 :-)

@nao-pon
Copy link
Member

nao-pon commented Oct 5, 2013

configのデフォルトを修正
 デフォルトを いいえ に
  ページナビゲーションを表示する
  印刷画面へのリンクを表示する
  友達に紹介するリンクを表示する
  tellafriendモジュールを利用する
  投票機能を利用する

この件は、慎重になります。デフォルトが OFF だと、その機能があることに気付かない可能性もあるので、より一般的にするには、どうすればいいかを 考察したいです。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants