Opened 13 years ago
Closed 3 years ago
#14781 closed feature request (wontfix)
Allow upload of PHP files as plugin
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Severity: | normal | Version: | |
| Component: | Upgrade/Install | Keywords: | has-patch needs-unit-tests |
| Focuses: | administration | Cc: |
Description
Currently allowed are only ZIP packages. Plugins that consist of a single PHP file are not supported. It would be handy if uploading a single PHP file is an option.
Attachments (4)
Change History (21)
#2
@
13 years ago
Workaround: Zipping the single one PHP file (not having any directory attached) and uploading it, does the job.
#3
@
13 years ago
- Component changed from General to Upgrade/Install
- Type changed from defect (bug) to enhancement
#4
follow-up:
↓ 5
@
13 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to Future Release
- Type changed from enhancement to feature request
Seems like a relatively sane idea, Checks would need to be performed obviously to ensure that the file contains proper plugin headers.
#5
in reply to:
↑ 4
@
13 years ago
Replying to dd32:
Seems like a relatively sane idea, Checks would need to be performed obviously to ensure that the file contains proper plugin headers.
Shouldn't we do the exact checks on the unpacked zip as well?
#6
@
13 years ago
Shouldn't we do the exact checks on the unpacked zip as well?
They are done AFAIK, However skipping the unzip process would bypass that section of code.
#7
@
13 years ago
- Keywords needs-testing added
I've attached a patch that fixes this issue functional-wise. Now you can upload a single PHP file or ZIP archives.
The thing that is not done is taking care over the texts and user messages.
Also needs testing.
#10
@
9 years ago
- Keywords needs-refresh added; has-patch removed
Seems like a relatively good idea. Patch needs a refresh though
#13
@
6 years ago
I refreshed the patch looking about how to do unit tests.
Seems that WP_Upgrader class doesn't have unit test so someone can help about?
I cannot find also an unit test for the actual code that check if the zip upload is working.
#14
@
3 years ago
- Keywords needs-refresh removed
14781.4.diff Refreshed patch so that it cleaning applies.
I did add 1 thing to the refreshed patch: if rename() fails, it sets $result to a WP_Error instance. Without this, the update would appear to succeed when it actually hasn't.
As @dd32 mentions, I think the patch should be extended to check that the uploaded single PHP file actually has plugin headers. That is not yet included in the patch.
I'd also say that I'm not sure that creating a sub-directory and putting the single PHP in there is right thing to do. Since core allows single file plugins to exist directly in WP_PLUGIN_DIR, if this ticket moves forward that might be what should happen.
But since at the moment I'm just triaging tickets in this component and refreshes patches where needed, I won't bother investigating that possibility at this time.
This ticket was mentioned in Slack in #core-auto-updates by afragen. View the logs.
3 years ago
This ticket was mentioned in Slack in #core-auto-updates by francina. View the logs.
3 years ago
#17
@
3 years ago
- Milestone Future Release deleted
- Resolution set to wontfix
- Status changed from new to closed
After reviewing the ticket in the month of May 2021, the people working on this component decided to close the ticket. The developer hub has some best practices for plugin format and those should be followed: https://developer.wordpress.org/plugins/plugin-basics/best-practices/#folder-structure
Thank you everyone for the discussion over the years.
Related: #9757