XoopsX / legacy

XOOPS Cube Legacy base repository
15 stars 11 forks source link

Primary key has many fields #51

Closed bluemooninc closed 10 years ago

bluemooninc commented 11 years ago

プライマリーキーが複数ある場合に対応したlegacyハンドラです。ご評価ください。

hamaco commented 11 years ago

ちょっとソース見ただけですが、$mPrimary = 'uid, item_id'; の時に間違えて、配列で渡さずに get( $uid ) とかしちゃった場合にエラー出してくれた方がデバッグしやすくて良いかも?

あと、折角バージョン管理システム使ってるので、古いコードコメントアウトせずに削除してしまって良いんじゃないでしょうか?

bluemooninc commented 11 years ago

コメント有り難うございます。エラー処理考えてみます。

nao-pon commented 10 years ago

CorePack ブランチは stable にする直前の検証用に使用することにしましたので、一旦クローズします。 再度リクエストする場合は、同じブランチを xoopscube/master と XoopsX/pilot に対して行なって頂くようお願い致します。