Ticket #425 (closed defect: fixed)

Opened 6 months ago

Last modified 5 months ago

Simple Media Silo Plugin: Plugin activated even upon error during activation

Reported by: jaypipes Owned by: arthus
Priority: major Milestone: 0.5
Component: Plugins Version: SVN
Keywords: media silo,plugin,has_patch Cc:

Description (last modified by michaeltwofish) (diff)

Marking as defect since this can cause a lot of headaches for users of this essential plugin.

On activation, if the user/ directory is not writeable, a very quick popup message is displayed saying so, but the plugin still goes into the activated plugin list. It should NOT do so, as the non-power user will assume that the plugin will work just fine then. Instead, a longer-lasting error message or instruction area should explain how to fix the permissions, and the plugin should remain in the de-activated list.

Duplicates: 243

Attachments

users.patch (476 bytes) - added by arthus 5 months ago.
files.patch (1.7 kB) - added by arthus 5 months ago.

Change History

Changed 6 months ago by ringmaster

It might be worthwhile to allow the plugin to activate even if the directory is not writeable, but also provide an option to specify the location of the directory to serve files from.

A warning could be displayed in the silo itself when an upload is attempted.

Changed 5 months ago by miklb

  • milestone changed from Undetermined to 0.5

Changed 5 months ago by arthus

  • owner set to arthus
  • status changed from new to assigned

Changed 5 months ago by arthus

Changed 5 months ago by arthus

  • keywords silo,plugin,has_patch added; silo,plugin removed

The patch will stop the plugin from being activated if the files directory isn't writable. Unfortunately, it also doesn't display an error message due to the limitations of the plugin API.

Changed 5 months ago by arthus

Changed 5 months ago by arthus

Sorry about that.... I accidentally uploaded the wrong patch. (I really have too many of these things sitting around.)

Changed 5 months ago by ringmaster

  • status changed from assigned to closed
  • resolution set to fixed

In r2185.

Changed 5 months ago by michaeltwofish

  • description modified (diff)
Note: See TracTickets for help on using tickets.