Skip to content

Convert civimix-schema@5 from PHAR to subdir#387

Merged
totten merged 3 commits intomasterfrom
master-phar-extract
Jan 25, 2025
Merged

Convert civimix-schema@5 from PHAR to subdir#387
totten merged 3 commits intomasterfrom
master-phar-extract

Conversation

@totten
Copy link
Copy Markdown
Owner

@totten totten commented Jan 24, 2025

Overview

Improve compatibility with Backdrop. Note that:

  • Backdrop CLI allows you to read PHAR files. So automated civix testing with civimix-schema@XX.phar had passed on Backdrop. Great, but...
  • Backdrop Web UI completely blocks usage of PHAR files. 🤦

Switching from PHAR to subdirectory works-around it.

Before

When using EFv2 with civimix-schema@5, the backport is created as a PHAR file.

After

When using EFv2 with civimix-schema@5, the backport is created as a subdirectory.

The problem here is that Backdrop compatibility:

* Deceptively, CLI tests work with PHAR on Backdrop.
* In reality, with regular web-requests, Backdrop disables support
  for PHAR files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant