msg# 1
GIJOE

登録日: 2006-3-20
投稿数: 3708
bulletin2とっても良いモジュールなのですが、細かな「要修正」がそれなりに溜まってきていて、バージョン管理がかなり面倒な状況になってしまってますね。
suinさん自身は、現在留学生活をとても楽しんでいるようなので、それに水を差すような真似はしたくないですし。
というわけで、suinさんにとっても利用者にとっても、状況を把握しやすいように、ここにまとめてみたいと思います。
とりあえず、suinさんオリジナルとしては最新版である 2.04 ベースで、Cube 2.1対応に必要なパッチは2つ。
http://homepage.mac.com/minahito/.Public/BulletinToA4.ziphttp://www.peak.ne.jp/support/xoops/bulletin-2.04patch4cube2.1beta.tar.gzどちらも2.04に上書きします。(同じファイルはいじってないので、順番は前後しても大丈夫です)
あと、だいぶ前ですが、boobeeさんの質問。これ、再現しました。
http://www.xugj.org/modules/plzXoo/index.php?action=detail&qid=702.03 -> 2.04 でのエンバグですね。
(うちのは2.03だったことに今頃気づきました。2.04に上げた途端に再現してます)
include/storyform.inc.php 81行目付近
// If user have right to use HTML
if( $gperm->group_perm(4) ){
$html_checkbox = new XoopsFormCheckBox('', 'html', $story->getVar('html') );
$html_checkbox->addOption(1, _MD_USE_HTML);
$option_tray->addElement($html_checkbox);
$br_checkbox = new XoopsFormCheckBox('', 'br', $story->getVar('br') );
$br_checkbox->addOption(1, _MD_USE_BR);
$option_tray->addElement($br_checkbox);
}
この、br関連を外側に追い出せばOKでしょう。
(確認してませんが)
// If user have right to use HTML
if( $gperm->group_perm(4) ){
$html_checkbox = new XoopsFormCheckBox('', 'html', $story->getVar('html') );
$html_checkbox->addOption(1, _MD_USE_HTML);
$option_tray->addElement($html_checkbox);
}
$br_checkbox = new XoopsFormCheckBox('', 'br', $story->getVar('br') );
$br_checkbox->addOption(1, _MD_USE_BR);
$option_tray->addElement($br_checkbox);
あと、このファイルの20行目。
$form->addElement(new XoopsFormText(_MD_TITLE, 'title', 50, 80, $story->getVar('title', 'f') ), true);
この80を255にしていただけるととっても嬉しいです。
(複数言語でタイトルを書くと、80文字じゃとっても足りません)
最後に、trust_path側のmain/rss.php。
$conetent = $article->getVar('hometext') . $article->getDividedBodytext() ;
(snip)
'content' => xoops_utf8_encode($conetent)
これ、XMLエンティティ違反になります。
私は、content要らないので、この部分をコメントアウトしてます。
-------------
2007/1/6 追記
http://www.peak.ne.jp/support/xoops/bulletin-2.04-hacked.zipとして、一通りパッチ済のアーカイブを用意してます。
上の手順はスキップして、これをいきなりインストールする、という形で(多分)OKです。
投票数:6
平均点:8.33
msg# 1.1
余力がないので現時点では構想だけになってしまうのですが、どこかのリポジトリにあげさせてもらってパッチあてとバグフィックス、メッセージカタログのコントリビュート対応をしていきたいなというのはありますね > Bulletin2
この3点に絞ればフォークほどのことにはならないと思いますし、よりいっそう勉学に励んでいただけるかも。
投票数:4
平均点:5.00
msg# 1.2
GIJOE

登録日: 2006-3-20
投稿数: 3708
news1.1由来の「盲腸」
サブメニューを消す。
trust_path側のxoops_version.php
82行目からの4行。
$modversion['sub'][1]['name'] = constant($constpref.'_SMNAME1');
$modversion['sub'][1]['url'] = 'index.php?page=submit';
$modversion['sub'][2]['name'] = constant($constpref.'_SMNAME2');
$modversion['sub'][2]['url'] = 'index.php?page=archive';
ここを削除またはコメントアウト。
投票数:1
平均点:10.00
msg# 1.3
本日,Cube の β4 と Bulletin2 を入れたのですが,news からのデータコンバートで "404 not found" になってしまいました。
コンバートの確認画面(index.php?op=convert)の「送信」ボタンを押すと,
/bulletin/admin/admin/index.php にアクセスしようとしていました。
そのため,GIGEO さんのところのパッチ
http://www.peak.ne.jp/support/xoops/bulletin-2.04patch4cube2.1beta.tar.gzの index.php(TRUST_PATH 側)の340行目
xoops_confirm(array('op' => 'convert', 'ok' => 1), 'admin/index.php', _AM_DO_YOU_CONVERT);
で,'admin/' が余計なのではないかと思います(ここを削ったら上手くいきました)。
※あとたぶん,401行目も同様。
ご確認頂けますでしょうか。
投票数:2
平均点:10.00
msg# 1.3.1
GIJOE

登録日: 2006-3-20
投稿数: 3708
確かに私のミスっぽいですね。修正しておきます。
bulletin2も今のままだと、インストールが面倒ってことになりかねないので、自分のためにも、SVNに入れて管理しようかとは思ってます。
# GIGEOではなく、GIJOEです
投票数:1
平均点:10.00
msg# 1.3.1.1
GIJOE

登録日: 2006-3-20
投稿数: 3708
いろいろ葛藤もあったのですが、bulletin2をSVNに入れて、自前管理することにしました。
各種パッチのあたった状態のアーカイブを
http://www.peak.ne.jp/support/xoops/bulletin-2.04-hacked.zipとして用意してます。
オリジナルとの差分は、berliosのSVNからどうぞ。
http://developer.berlios.de/projects/peakxoops/オリジナルに極力手は加えずに、問題のある動作だけをピンポイントで修正する、というコンセプトです。
news1.1からのインポートについても修正してあります。
投票数:1
平均点:10.00
msg# 1.3.1.1.1
GIJOE さん、bulletin2 のメンテナンス、おつかれさまです。
Amethyst Blue へもコメントをつけておられましたね。しばらくの間は、あっちに書き込んでもらうよりも、XUGJ や
ひよこむに誘導した方がいいかもしれませんね……。
投票数:1
平均点:10.00
msg# 1.4
toshi

登録日: 2006-12-12
投稿数: 135
Cube Legacy 2.1 RC で bulletin2 を利用した場合、一部のブロックで使用言語が正しく反映されない場合があるようです。
具体的には「本日のトップニュース」ブロックですが、使用言語を Japanese にしていても、該当するニュースがない場合
There is no top story today, yet.
と英語の表示になってしまいます。
そこで、xoops_trust_path/modules/bulletin/blocks.php の7行目あたり、
// language files
global $xoopsConfig; // <== この行を追加
$language = empty( $xoopsConfig['language'] ) ? 'english' : $xoopsConfig['language'] ;
こんな感じで、global $xoopsConfig; の1行を入れると、「本日のトップニュースはありません」と日本語で表示できるようになったのですが、PHPは初心者のため、この解決方法が一番いいかどうか自信がありません。
他に解決方法がありましたら、ご教示いただけませんでしょうか。よろしくお願いいたします。
投票数:1
平均点:10.00
msg# 1.4.1
GIJOE

登録日: 2006-3-20
投稿数: 3708
toshiさん、こんにちは。
ああ、なるほど。$xoopsConfigって、X2だとなぜか利用できるんですね。
とりあえずglobalでまったく問題ありません。
私の管理しているアーカイブの方では、他のD3モジュールと同じ記述に変更しておきました。
投票数:1
平均点:10.00
msg# 1.4.1.1
toshi

登録日: 2006-12-12
投稿数: 135
GIJOEさん、ありがとうございます。
問題はないということで安心しましたが、GIJOEさんの方でも早速修正していただいたようで、管理されているアーカイブのファイルに差し替えさせていただきました
投票数:4
平均点:7.50
msg# 1.3.1.1.2
GIJOE

登録日: 2006-3-20
投稿数: 3708
引用:
オリジナルに極力手は加えずに、問題のある動作だけをピンポイントで修正する、というコンセプトです。
…と書いておいてナニですが、D3モジュールだと、xoops_breadcrumbs への自動対応がイマイチ機能しないので、実装しちゃいました。
http://www.peak.ne.jp/support/xoops/bulletin-2.04-hacked.zipあとは、d3forumを片付ければ、主要なモジュールは一通り対応ってことになる…かな?
(TinyDみたいな深さのないモジュールなら自動対応で十分だし、mylinks系の階層カテゴリーを持つモジュールも一通りリストアップしたので)
投票数:2
平均点:10.00
msg# 1.5
GIJOE

登録日: 2006-3-20
投稿数: 3708
書き忘れてましたが、005と006のCHANGELOG
もうパッチで提供できるほどの変更量じゃなくなったので、svnでdiffとってください。
005
xoops_breadcrumbs 規格導入。
ぜひとも、xoops_breadcrumbsが浸透してほしい、という政治的な理由のため(笑)
年月日コントローラがおかしかったので修正
006
トピック選択のtypo fix
トピックの階層順次絞り込み
階層サブメニュー実装
この1番目は判りやすいtypoでした。
$xt->makeTopicSelBox($none=0, $seltopic, $selname, $onchange);
というわけで、xoops_breadcrumbs と 階層サブメニューの両対応モジュールは計3つに!
投票数:1
平均点:10.00
msg# 1.6
GIJOE

登録日: 2006-3-20
投稿数: 3708
007 2007-03-19
RSSのタイムゾーンがおかしかったのを修正
formatTimestamp( $timestamp , 'rss' ) ;
これはつかっちゃ駄目ですね。(今頃気づいた)
サーバ->ユーザの時差計算をしたものに、サーバの時差表記が付与されるので、完全にアウトです。
include/functions.php を書いた人は何を考えていたんだろう?
ともあれ、RSS用の時間表記をするなら、素直にこうです。
date( 'r' , $timestamp ) ;
投票数:1
平均点:10.00
msg# 1.7
suin

登録日: 2006-3-23
投稿数: 99
久しぶりに書き込みます。
引用:
GIJOEさんは書きました:
bulletin2
とっても良いモジュールなのですが、細かな「要修正」がそれなりに溜まってきていて、バージョン管理がかなり面倒な状況になってしまってますね。
suinさん自身は、現在留学生活をとても楽しんでいるようなので、それに水を差すような真似はしたくないですし。
というわけで、suinさんにとっても利用者にとっても、状況を把握しやすいように、ここにまとめてみたいと思います。
XUGJに丸投げするかたちになってしまい、本当に申し訳ございませんm(_ _"m)ペコリ
留学が終わり帰国してそろそろ1ヶ月になります。
帰国すればXOOPSに少し集中ができるかなと思ったのですが、どうも今年は韓国語とは別に大学院用の外国語をやらなければいけないみたいですT-T
今年も活発に参加できるか解りませんが、どうぞ暖かく見守っていただければと思います<(_ _)>
投票数:2
平均点:10.00
msg# 1.7.1
fabi

登録日: 2006-3-14
投稿数: 1536
suinさん、おかえりなさい!
投票数:1
平均点:10.00
msg# 1.7.2
前の投稿
-
次の投稿
|
親投稿
-
子投稿なし
|
投稿日時 2007-3-22 4:07
GIJOE

登録日: 2006-3-20
投稿数: 3708
帰国しているのはmixiとかで知ってはいましたがあらためて、
「お帰りなさ〜い」
どう考えても、勉強の方が大事ですから、気にしなくていいですよ〜
投票数:4
平均点:10.00
msg# 1.6.1
GIJOE

登録日: 2006-3-20
投稿数: 3708
前回のをさぼってしまったので、2回分まとめて報告。
008 (2007-04-23)
XC Legacy2.1でモジュールアップデートしたときにエラーが出るのを修正してます
009 (2007-05-08)
Starckさんのwaitingプラグインを取り込みました。
例の言語定数オーバーライドシステムを導入してます。
それにともなって、altsys必須となってしまいました。
今気づきましたが、bulletinのブロック用言語定数は、固定になってますね。
これだと、モジュール毎にブロック用言語定数を切り替えることができません。
本当は$constprefを使った形にしたいのですが、block内からmainの言語定数を使っている部分もあって簡単じゃないのでやめました。
あと、koreanとtchineseがなぜか変なエラーを起こすので、いったん削除しました。
5c問題かなあ?
ついでにjapaneseutfを追加しておきました。
投票数:1
平均点:10.00
msg# 1.6.1.1
GIJOEさん Suinさん
日頃bulletin2を利用させて頂いております。
要望で申し訳ありませんが、
bulletin2にWYSIWIGエディタ(SPAW,FCKEitor)に対応させる事は難しいでしょうか。
現在、社員向けイントラサイトでbulletin2を利用させていただいているのですが、いかんせん50代のおばちゃんが社内広報などを入力しているので、、、できたらFCKEditorを利用して入力させたいと考えています。
(現在はPICOで入力しHTMLソースを貼り付けてなんとか使ってもらっています)
誠に勝手な要望ですが検討して頂けたら幸いです。
投票数:1
平均点:10.00
msg# 1.6.1.1.1
Kaz

登録日: 2006-3-20
投稿数: 228
引用:
いかんせん50代のおばちゃんが社内広報などを入力しているので、、、できたらFCKEditorを利用して入力させたいと考えています。
(現在はPICOで入力しHTMLソースを貼り付けてなんとか使ってもらっています)
横レスぽいですが、すいません^^
「HTMLソースを貼り付ける」というのは、逆にとても高度なんじゃないかという気がするんですが、具体的に、どういうとこが欲するところなんでしょうか?
素朴にギモンです^^
投票数:1
平均点:10.00
msg# 1.6.1.1.2
GIJOE

登録日: 2006-3-20
投稿数: 3708
技術的に簡単すぎて面白くないために特にやっていませんでしたが、もったいぶる理由もないので011でつけておきました。
010 (2007-05-12)
MySQL5で問題になるというFULL TEXT KEYを削除しました。
011 (2007-05-15)
FCKeditor on XOOPS を使う設定を追加
なお、HTMLを許可していないユーザには、設定でONになっていても、従来通りのフォームしか表示されません。
(脆弱性うんぬんではなく、単に意味がなく、誤解を招きやすいため)
Wysiwygエディタを使うとか使わない以前に、不特定多数へのHTML許可はやめてくださいね。
(当たり前ですが)
投票数:2
平均点:10.00
msg# 1.6.1.1.1.1
Kazさんこんにちわ
「HTMLソースを貼り付ける」のはFCKEditorですと簡単でWYSIWIGモードで字飾りなどをして完成させそのごHTMLボタンがあるのでソースを表示します。
そうするとHTMLタグが付いたソースが表示されるのでそれをコピーしBulletinに貼り付けていました。(勿論HTMLタグを有効にしてです)
投票数:1
平均点:10.00
msg# 1.6.1.1.2.1
GIJOEさんこんにちわ
FCKEditor対応有難うございます!!
早速検証し導入したいと思います。
>>Wysiwygエディタを使うとか使わない以前に、不特定多数へのHTML許可はやめてくださいね。
標記件承知致しました、WysiwygエディタやHTMLタグは社内イントラネットのみで利用します
本当に有難うございました。 m(_~_)m
投票数:1
平均点:10.00
msg# 1.6.1.1.1.1.1
Kaz

登録日: 2006-3-20
投稿数: 228
なるほど〜。文字修飾ですか。
確かに,通常の投稿フォームではやりづらいですよね。
投票数:1
平均点:10.00
msg# 1.6.1.1.1.1.1.1
toshi

登録日: 2006-12-12
投稿数: 135
明けましておめでとうございます。本年もよろしくお願いいたします。
現在 bulletinHD 2.1 を使用していますが、掲載期限を過ぎた記事がサイト内検索の検索対象となり、クリックすると「選択されたニュース記事は存在しません」と叱られてしまいます。
xoops_trust_path/modules/bulletin/search.php の 16〜20行目あたりを
if( $showcontext == 1 && function_exists('search_make_context')){
$sql = "SELECT storyid,uid,title,published,expired,hometext,bodytext,html,smiley FROM ".$xoopsDB->prefix($mydirname."_stories")." WHERE published > 0 AND published <= ".time()." AND (expired = 0 OR expired >= ".time()." )";
}else{
$sql = "SELECT storyid,uid,title,published,expired FROM ".$xoopsDB->prefix($mydirname."_stories")." WHERE published > 0 AND published <= ".time()." AND (expired = 0 OR expired >= ".time()." )";
}
などと修正するのがいいのかなと思っていますが、いかがでしょうか。
投票数:0
平均点:0.00
msg# 1.6.1.1.1.1.1.1.1
GIJOE

登録日: 2006-3-20
投稿数: 3708
toshiさん、こんにちは。明けましておめでとうございます。
引用:
現在 bulletinHD 2.1 を使用していますが、掲載期限を過ぎた記事がサイト内検索の検索対象となり、クリックすると「選択されたニュース記事は存在しません」と叱られてしまいます。
なるほど、オリジナルバグですね。
さっそく修正しました。
ご報告ありがとうございました
投票数:4
平均点:5.00