Status report always says "Not writable" [#1248514]

diff -wBNru css_injector/css_injector.install modules/css_injector/css_injector.install
--- css_injector/css_injector.install   2011-02-24 21:08:11.000000000 -0500
+++ modules/css_injector/css_injector.install   2011-08-13 15:47:34.000000000 -0400
@@ -85,7 +85,7 @@
       'title' => t('CSS Injector directory writable'),
       'description' => $status == REQUIREMENT_OK ? t('CSS Injector Directory %dir is writable', array('%dir' => $dir)) : t('Directory %dir is not writable', array('%dir' => $dir)),
       'severity' => $status,
-      'value' => t('Not writable'),
+      'value' => $status == REQUIREMENT_OK ? t('Writable') : t('Not writable'),
     ),
   );
   return $requirements;

Comment	File	Size	Author
#2	css_injector-requirement-should-check-status-1248514-2.patch	1004 bytes	makangus
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rfay

English

Palisade, CO, USA

CreditAttribution: rfay commented 14 August 2011 at 01:52

You'll need to give a better description than this to get something worked on (and patches are welcome - it looks like you're capable of making one... - maybe you just intended to attach instead of pasting something in?). Please give a complete description of your issue. Please take a look at http://drupal.org/node/73179 and http://drupal.org/node/1155816

I can confirm what I think is your complaint though. Looking forward to a better description and a patch. Thanks for your contribution.

Comment #2

makangus CreditAttribution: makangus commented 7 September 2012 at 04:46

Status:

Active

» Needs review

File	Size
css_injector-requirement-should-check-status-1248514-2.patch	1004 bytes

I was bored so I decided to roll a patch. I think the bug report from fizk is about "CSS Injector directory writable" is always showing "Not writable" on the status report page. Attached is the patch against the 7.x-1.x branch that checks the status and put it either "Writable" or "Not writable". The code was provided by fizk.