simplesamlphp / simplesamlphp/simplesamlphp-module-metarefresh
writeMetadataPdo surplus configuration
まだ誰も着手していません。
- 主要言語
- PHP
- スター
- 9
- フォーク
- 19
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
There's a Too many arguments to function __construct(). 2 provided, but 1 accepted. warning being picked up in MetaLoader.php:675
The constructor on Metadata\MetaDataStorageHandlerPdo accepts one config parameter but then doesn't do anything with it anyway, calling Database::getInstance(); instead.
My inclination was to drop passing the config in the metarefresh module entirely but not sure if that's the right approach.
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
MetaLoader.php:675 から始めて Metadata\MetaDataStorageHandlerPdo の構築を調べ、そのクラスのコンストラクターおよび Database::getInstance() の使用方法と比較します。余分な設定を削除すべきか、一貫した方法で処理すべきかを判断し、コンストラクターの警告が表示されなくなったことを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- php
- 領域
- database
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 42/100