simplesamlphp / simplesamlphp/simplesamlphp-module-discopower

Symlinks don't install correctly when Composer falls back on the PHP zip library

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

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

主要言語
PHP
スター
5
フォーク
11
PR マージ指標
30日以内にマージされた PR はありません

説明

I couldn't figure out why the Discopower module wasn't installing correctly inside my Docker image of SSP. The symlinks under modules/discopower/public/assets/components were being installed as text files containing the path where they would have pointed had they been symlinks. This obviously didn't work so great when my browser tried to load the Jquery Javascript for Discopower.

After further investigation, I tried installing unzip into my Docker image before running composer, and that fixed it. I now have symlinks instead of text files. Clearly, the PHP zip library doesn't understand symlinks.

If possible, installation of this module should require that the unzip or 7z binary be installed on the system and not just the PHP zip library.

If that's not possible, the documentation should at least include this requirement for proper installation.

Ideally, these symlinks wouldn't be necessary, and the disco.twig code that loads these components would point to the files directly instead of the symlinks inside of the module.

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

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

はじめの一歩

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

調査の方向性

まず、説明されている Docker セットアップで Composer のインストールを再現し、PHP の zip ライブラリと unzip または 7z を比較します。modules/discopower/public/assets/components と、これらのアセットを読み込む disco.twig のコードを調査します。インストールの動作または文書化されたシステム要件が symlinks を明確に扱っていれば完了です。

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

評価

技術スタック
php
領域
build-system
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
35/100

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

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