simplesamlphp / simplesamlphp/simplesamlphp-module-metarefresh

writeMetadataPdo surplus configuration

オープン
#47 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

主要言語
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.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

MetaLoader.php:675 から始めて Metadata\MetaDataStorageHandlerPdo の構築を調べ、そのクラスのコンストラクターおよび Database::getInstance() の使用方法と比較します。余分な設定を削除すべきか、一貫した方法で処理すべきかを判断し、コンストラクターの警告が表示されなくなったことを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php
領域
database
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。