Visiting admin/reports/status is not helping fixing config options.

CommentFileSizeAuthor
#1 plupload-requirements-1110830-1.patch965 bytesclemens.tolboom

Comments

clemens.tolboom’s picture

clemens.tolboom
Primary language Dutch
Location Groningen, 🇳🇱/🇪🇺
CreditAttribution: clemens.tolboom commented
Status: Active » Needs review
FileSize
plupload-requirements-1110830-1.patch965 bytes
David_Rothstein’s picture

David_Rothstein CreditAttribution: David_Rothstein commented
Status: Needs review » Fixed

Thanks for the patch!

I committed this with some minor cleanup and changes:
http://drupalcode.org/project/plupload.git/commit/cfa1f88

The main changes I made were:

  1. Made the strings properly translatable.
  2. Only printed the message in the case there was an error. (Since we don't currently detect library version info, I didn't want to print a message saying everything is OK when it might be the case that someone is using an old, out-of-date version of the library. Maybe we can try adding the REQUIREMENT_OK message back in if/when we can also print the current version.)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit cfa1f88 on master, 7.x-2.x, 8.x-1.x by David_Rothstein: 
    Issue #1110830 by clemens.tolboom and David_Rothstein: Add a...