xoopsx / legacy Goto Github PK

This project forked from xoopscube/legacy

XOOPS Cube Legacy base repository

CSS 0.97% HTML 11.74% PHP 75.18% Smarty 0.63% JavaScript 11.07% Shell 0.01% ColdFusion 0.03% Perl 0.02% Hack 0.01% TSQL 0.36%

legacy's People

Contributors

Stargazers

Watchers

Forkers

nao-pon argon358 nouphet naao soft-studio ohsepoa domifara ihackcode mambax7 sohgo hodakamio

legacy's Issues

[user] enable to change default value

messageモジュールのデフォルト値オーバーライドと似ていますが、Userモジュールのイベント通知の送信先が、デフォルトだとPMなんですが、デフォルトをE-mailにしておきたい場面もありますので、　そちらもオーバーライドできるようにしたいです。

[message] enable to change default value

messageモジュールで、　各人の設定を行わないデフォルト状態の挙動を変えたい時、
https://github.com/xoopscube/legacy/blob/master/html/modules/message/class/handler/Settings.class.php#L9

public function __construct()
内の初期値を変更して使っています。
今のままですと、X-updateでlegacyをアップデートすると上書きされて元に戻ってしまうので、何らかの方法でオーバーライド保存できるようにしたいです。

Session can not be continued in a particular environment

Session data might not be saved. For example, when APC is enabled. The reason is because the object is destroyed before the session writing at the time of closing PHP.

ref. http://xoops.hypweb.net/modules/forum/index.php?topic_id=3517

XCL が送信するメールの送信元アドレスをしていできるように

ユーザー登録時、パスワード紛失時、イベント通知時などで送信されるメールアドレスが、管理者メールアドレスに固定されているので、送信専用アドレスなどを持つ場合には、それを指定できるようにしたい。

mContext->mModule is not null on the top page

ゲスト以外でトップページにアクセスするとつぎのエラーが出ます。

Fatal error: Call to undefined method stdClass::isActive() in
 ドキュメントルート/modules/legacy/kernel/Legacy_Controller.class.php on line 958

環境は、さくらレンタルサーバー、xoopsX/pilot、php7.2です。
ちなみに、トップのときのmRoot->mContextの内容は、

["mModule"]=>
    object(stdClass)#146 (1) {`
        ["mXoopsModule"]=>
            NULL
    }
["mXoopsModule"]=>
    NULL

なので、該当のログインユーザーの場合の行：

if ($this->mRoot->mContext->mModule != null && $this->mRoot->mContext->mModule->isActive()) {

が冒頭のエラーを吐く感じです。とりあえず、

if ($this->mRoot->mContext->mModule instanceof Legacy_ModuleAdapter && $this->mRoot->mContext->mModule->isActive()) {

として回避しましたが、$this->mRoot->mContext->mModuleがnullかどうかを判定する箇所は他にもあると思われますので、ベスト解ではないと思います。

Notice: date_default_timezone_set(): Timezone ID '9' is invalid


( ! ) Notice: date_default_timezone_set(): Timezone ID '9' is invalid in /Users/suin/Dropbox/Suin/Projects/GitHub/xoops-commentin/html/install/include/makedata.php on line 69
Call Stack
#   Time    Memory  Function    Location
1   0.0001  245312  {main}( )   ../index.php:0
2   0.0062  312664  include( '/Users/suin/Dropbox/Suin/Projects/GitHub/xoops-commentin/html/install/custom/install_insertData_theme.inc.php' )  ../index.php:98
3   0.0066  340912  require_once( '/Users/suin/Dropbox/Suin/Projects/GitHub/xoops-commentin/html/install/wizards/install_insertData.inc.php' )  ../install_insertData_theme.inc.php:3
4   0.0279  1148512 make_data( )    ../install_insertData.inc.php:52
5   0.0299  1155560 date_default_timezone_set ( )   ../makedata.php:69

管理画面のブロック管理で、ブロック一覧から　X2互換の使用先モジュールでの抽出を追加したい

Installer

とりあえずメモです。Parse Error が出る。

たぶん、この修正です。
f3e274a#diff-c553d3e31f199a05ad82def21c71007b

legacy moduleCore Bug.

modules\legacy\admin\blocks\AdminSideMenu.class.php
82行目

$t_module =& $moduleHandler->getByDirname(xoops_gethandler('dirname'));
$t_module =& $moduleHandler->getByDirname(xoops_gethandler('legacy'));

まりさん経由でこぴぺです。

テーマ下テンプレートの default 対応（resource.db.phpの換装）

https://www.facebook.com/groups/313506525343887/permalink/468807433147128/?comment_id=469457539748784&offset=0&total_comments=30

澤田直樹 Bootstrap対応テーマの件で。よく分かってないのですが、テーマ名規則でで親テーマ・子テーマを定義できるようにして、子テーマ使用時、親テーマのテーマ下テンプレートを利用できるような感じで対応できますか？
これは、モバイル対応にも使えるかも。
6月22日 10:21

沖野直毅 resource.db.php　を換装する、ということですね。了解です。
6月23日 9:28

澤田直樹 hoge_fuga というテーマのテーマ下テンプレートがない場合、hoge_default のテーマ下テンプレを探す resource.db.php を書いてみました。
https://gist.github.com/2992451
6月26日 10:17

澤田直樹 Tracker: Feature Requests : Legacy に上げました。
https://sourceforge.net/tracker/?func=detail&aid=3537989&group_id=159211&atid=943472
6月26日 10:53

澤田直樹 bootstrap_default に Bootstrap 対応テンプレを置いておけば、bootstrap_a でも bootstrap_b でも bootstrap_default のテーマ下テンプレが使用されるという算段です。
6月26日 11:00

沖野直毅澤田さん
あっちこっちやることが多くて大変なんですが、　これ、コアより先にCorePackに組みこんじゃいませんか？

丁度今、XUGJにテンプレート標準化したいです、という書き込みがあって、氷川さんがBootStrap化の考えを示唆したところで、これは丁度良いタイミングですね。^^
http://www.xugj.org/modules/d3forum/index.php?topic_id=1230#post_id6794
7月1日 11:08

澤田直樹私はデザイン系に疎いので、よく分かっていないのですがこの仕様が bootstrap 対応に有用であるなら先行して導入する価値はあると思うので、いいと思います。
7月1日 11:13

澤田直樹 resource.db.php をもうちょっといじってみました。

https://gist.github.com/2992451

設定により、有効・無効と優先順位を指定できるようにしてみました。

設定は site.ini (site_default.ini) にて

[Smarty]
ResourceDiscoveryOrder=Theme,ThemeD3,ThemeDefault,ThemeDefaultD3,DbTplSet

のようにできるようにしてあります。site.ini に設定値がない場合は ”Theme,ThemeD3,ThemeDefault,ThemeDefaultD3,DbTplSet” になってます。
7月2日 23:44

pack2011_default theme & GNAVi map show size problem

pack2011_default テーマで、GNAViのマップが　CENTER LEFTな表示になってしまいます。　他のテーマにすると、正常表示されます。

syntax errorおきてる

2008-11-23 15:53 minahito           29319eb          * @param   string  $name   Filename
2008-11-23 15:53 minahito           29319eb          * @param   boolean $gzip   Use GZ compression
2008-11-23 15:53 minahito           29319eb          **/
2008-11-23 15:53 minahito           29319eb         function download($name, $gzip = true)
2008-11-23 15:53 minahito           29319eb         {
2012-02-23 07:34 kilica             7d5173b                 $file = $this->archiver->toTarOutput($name.$this->ext, $gzip);
2008-11-23 15:53 minahito           29319eb                 $this->_header($name.$this->ext);
2012-02-23 07:34 kilica             7d5173b                 header(\'Content-Type: application/x-tar\') ;
2012-02-23 07:34 kilica             7d5173b                 header(\'Content-Length: \'.floatval(@strlen($file))) ;
[blame] html/class/tardownloader.php - line 176 of 180 (97%)

diff --git a/html/class/tardownloader.php b/html/class/tardownloader.php
index 20d060c..057df6a 100644
--- a/html/class/tardownloader.php
+++ b/html/class/tardownloader.php
@@ -170,8 +170,11 @@ class XoopsTarDownloader extends XoopsDownloader
         **/
        function download($name, $gzip = true)
        {
+               $file = $this->archiver->toTarOutput($name.$this->ext, $gzip);
                $this->_header($name.$this->ext);
-               echo $this->archiver->toTarOutput($name.$this->ext, $gzip);
+               header(\'Content-Type: application/x-tar\') ;
+               header(\'Content-Length: \'.floatval(@strlen($file))) ;
+               echo $file;
        }
 }

XupdateInstallUtils.class.phpのSQL文がばぐってる

2012-01-28 04:56 naao               4c7a461         if(!$isNew)
2012-01-28 04:56 naao               4c7a461         {
2012-01-28 04:56 naao               4c7a461             return true;
2012-01-28 04:56 naao               4c7a461         }
2012-01-28 04:56 naao               4c7a461
2012-01-28 04:56 naao               4c7a461         if($autoLink)
2012-01-28 04:56 naao               4c7a461         {
2012-01-28 04:56 naao               4c7a461             $sql = sprintf(
2012-01-28 04:56 naao               4c7a461                 'insert into `%s` set (`block_id`,`module_id`) values (%d,0);',
[blame] xoops_trust_path/modules/xupdate/admin/class/installer/XupdateInstallUtils.class.php - line 409 of 1232 (33%)
commit 4c7a4616eb1d52efc3dbe20681fae37366b3a341
Author:     naao <naao@68c2734e-149c-420d-9c06-e9d97fbf3a51>
AuthorDate: Sat Jan 28 04:56:23 2012 +0000
Commit:     naao <naao@68c2734e-149c-420d-9c06-e9d97fbf3a51>
CommitDate: Sat Jan 28 04:56:23 2012 +0000

    prototype xoops_trust_path side

pack2011_default theme's image slider with bug

pack2011_default テーマの画像スライダが、正しく動作しないです。
１枚目→２枚目はＯＫ、　その後、２→３、３→１の時に、　３度ほど引っ掛かって戻りを繰り返したあと、　進む感じです。

change file name of download user data fine neme.

fix to '%s User data List.csv' to '%s_User_data_List.csv'

インストーラの改善 3クリックでインストールできるようにする

インストールが長い。

安定版ダウンロードで、corepack_20130605がダウンロードされる

http://www.xoopscube.net/modules/xpwiki/#wa5e9b19
にある「安定版ダウンロード」のリンク先
https://github.com/XoopsX/legacy/zipball/stable
でダウンロードされるのが、corepack_20130810でなく一つ前のcorepack_20130605になってるようです。

messageモジュールインストール時のpmモジュからのデータ移行にバグ

html/modules/message/admin/class/myInstaller.class.php 63行目から

while ($val = $db->fetchArray($result)) {
      $sql = sprintf($INBOX, $val['to_userid'], $val['from_userid'], mysql_real_escape_string($val['subject']), mysql_real_escape_string($val['msg_text']), $val['msg_time'], $val['read_msg']);
      $result = $db->queryF($sql);

      $sql = sprintf($OUTBOX, $val['from_userid'], $val['to_userid'], mysql_real_escape_string($val['subject']), mysql_real_escape_string($val['msg_text']), $val['msg_time']);
      $result = $db->queryF($sql);
      $num++;
    }

と、whileに渡してる$resultをループ内のqueryFで上書きしちゃうので、データ移行がしくじります。

while ($val = $db->fetchArray($result)) {
      $sql = sprintf($INBOX, $val['to_userid'], $val['from_userid'], mysql_real_escape_string($val['subject']), mysql_real_escape_string($val['msg_text']), $val['msg_time'], $val['read_msg']);
      $result2 = $db->queryF($sql);

      $sql = sprintf($OUTBOX, $val['from_userid'], $val['to_userid'], mysql_real_escape_string($val['subject']), mysql_real_escape_string($val['msg_text']), $val['msg_time']);
      $result3 = $db->queryF($sql);
      $num++;
    }

とすればpmのデータ読み取ってmessageモジュールに移行されました。

QuickManageBlock + 携帯対応レンダラーでのタイトル表示乱れ

QuickManageBlock + 携帯対応レンダラーの環境で、特にスマホ向け表示の場合にタグのタイトルの表示が乱れる。

QuickManageBlock を修正して携帯対応レンダラーでのレンダリング時は、編集・アンインストールリンクを、title ではなく contents に書き出すとよいと思います。

XOOPS_DB_TYPE に mysqli をセットすると文字化けする

こちらを参考に以下を試したところ文字コードが合っていないようで文字化けしました。

define('XOOPS_DB_TYPE', 'mysqli');

適当に mysqlidatabase.php に以下を追加したら直ったのですが、

<?php
$this->setCharset("utf8");

これをコアハックせずに正しく対応するにはどうしたらいいでしょうか。

また、これはxoops内部が他の文字コード（euc-jp？) で動いているということでしょうか。utf-8にデフォルトで対応する予定はあるでしょうか。

到達不能の変数を削除する

html/core/XCube_RenderSystem.class.php の114行目あたりの $type は要らないとおもいます。

2008-11-23 15:53 minahito           29319eb    │  
2008-11-23 15:53 minahito           29319eb    │         /**
2008-11-23 15:53 minahito           29319eb    │          * Return render-target type.
2008-11-23 15:53 minahito           29319eb    │          * @return int
2008-11-23 15:53 minahito           29319eb 110│          * @deprecated
2008-11-23 15:53 minahito           29319eb    │          */
2008-11-23 15:53 minahito           29319eb    │         function getType()
2008-11-23 15:53 minahito           29319eb    │         {
2008-11-23 15:53 minahito           29319eb    │                 return $this->getAttribute('legacy_buffertype', $type);
2008-11-23 15:53 minahito           29319eb 115│                 //return $this->mType;
2008-11-23 15:53 minahito           29319eb    │         }
2008-11-23 15:53 minahito           29319eb    │

User Profile Bug

Adding new user on Admin Panel is fail. Fix the bug. プロフィールモジュールで項目を追加している場合、管理画面からユーザを追加しようとするとユーザの追加が正常にできないのを修正。

invalid setup error : path / url check on php5.4.13

XOOPS_ROOT_PATHなど、内部にて変数が正常に展開されず、インストールを進めることが出来ませんでした。mainfile.phpを編集しても、同様のエラーが出ます。

-環境:
PHP 5.4.13
XoopsX-legacy-corepack_20130224-37-gd002ad2 (安定版)および Nightly 両環境にて確認
-URL
../install/index.php
-画面
パス・URLのチェック
NG 検知されたルートディレクトリのパスは、設定されたもの(XOOPS_ROOT_PATH)と異なります。
NG 設定されたルートディレクトリのパスは、ディレクトリではありません。
NG 設定されたURLは、不正な形式です。

XOOPS Cubeのディレクトリへのパス： XOOPS_ROOT_PATH
XOOPS CubeへのURL： XOOPS_URL
上記設定が正しい場合は、インストールを続けてください。
間違っている場合は、はじめからやり直してください。
または、mainfile.phpを直接編集して、このページを再読み込みしてください。

mysqlが絵文字に対応していない

こちらで対応していただいたコードを見て気付いたのですが、mysqlの文字コードが utf8_general_ci のようです。
これはスマホから絵文字を投稿されるとサービスが止まってしまう文字コードですので、 utf8mb4_general_ci にする必要があるかと思います。（MySQL, MariaDB）

対応する場合は、ここらへんをmysqlのバージョンに合わせて書き換えればいいでしょうか。

既存システムはどうすればいいかな。変換する方法はあるようですが、こういった場合はどう対応するのがxoops流でしょうか。まずは新規システムだけでも対応したいところです。

Image is old on before 2nd install step

インストール第2ステップに入る前の、管理者情報を入力する画面で表示される画像が、Pack2011のままなので、XOOPS Xのに差し替えたい。

php7.1でのインストール

こちらでよいかわかりませんが、対応いただけると助かります。

インストーラでエラーが出て完了できません。

Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP

Step 1はエラーが出ても、完了することができました。
Step 2は、進めませんでした。
phpは7.1です。

よろしくお願いします。

name case mismatch

Fatal error: Call to a member function find() on a non-object in modules\legacy\admin\class\ModuleInstallUtils.class.php on line 358

        $tplhandler = xoops_getHandler('tplfile');
        $delTemplates = null;
        
        $delTemplates =& $tplHandler->find($tplset, 'module', $module->get('mid'));

The $tplhandler in php $tplhandler = xoops_getHandler('tplfile');

should be:

php $tplHandler

smartyはデフォルトでhtmlエスケープ出力したほうがいい？

xoopsのsmartyは生の値？を出力するのがデフォルトなので、htmlタグをそのまま出力する場合は自前でescapeする必要があります。

例１：

<{$var | escape:'html'}>

これはセキュリティに問題を抱えたコードが生まれやすい仕様のため、

<?php
$xoopsTpl->default_modifiers = array('escape:"html"');

などとしてデフォルトをエスケープにして、あえてhtmlタグ等をそのまま出したい場合だけ

<{$var | smarty:nodefaults}>  // または no_enc など独自にわかりやすいものを作る

とする仕様が、（一長一短ではありますが）基盤となるCMSの仕組みとしてはいいように思います。

・・ということが社内で話題になったのですが、一般的には/xoopsの方針としてはどうなんでしょうか。（自分一人なら自動エスケープは使わないですが、セキュリティに弱いコーダーも入り交じるOSSの場では自動エスケープをデフォルトにしたほうがいいように感じます。）

可能であればどなたかのご意見を伺いたく思います。いかがでしょうか。

InnoDBへの対応状況、その課題を教えていただけますか

なかなかcloudに移行できない社内事情で、bare metal + galeraに移行したいのですがxoopsのMyISAMをどうしたものかと思っております。

どなたかInnoDBにしてみてあれがダメだった、これがどうしようもなかった的なことをご存じでしたら教えていただけませんでしょうか。