The documentation in the schema for the {files}.status column is wrong. In the file system/system.install,

'status' => array(
        'description' => t('A flag indicating whether file is temporary (1) or permanent (0).'),

should be changed into

'status' => array(
        'description' => t('A flag indicating whether file is temporary (0) or permanent (1).'),

The change is at line 594 in Drupal 6.3.

CommentFileSizeAuthor
#2 system_files_schema.diff515 bytesemok

Comments

robin monks’s picture

robin monks commented
Status: Needs review » Postponed (maintainer needs more info)

I'm sorry, but we need a patch to be able to review and eventually fix your issue. For information on creating a patch yourself (it's really easy, honest!) go to http://drupal.org/patch/create .

Thanks for reporting your issue!
Robin Monks
Drupal Testing/QA Volunteer

emok’s picture

emok commented
Version: 6.3 » 6.4
StatusFileSize
new system_files_schema.diff515 bytes

OK. Patch attached.

xjm’s picture

xjm
she/her
Primary language English
commented
Title: files.status documented with oppisite values » files.status documented with opposite values

This issue was patched in D7: #437822: Description of {files} status flag wrong way round

The documentation bug still exists as of D6.17, though. (I wasted a bit of time wondering what kept randomly eating some files that had been imported from D5... that'll teach me to use shortcuts.)

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Postponed (maintainer needs more info) » Closed (duplicate)

I've reopened that other issue to fix the documentation there, since it has a valid patch (for D7). Closing this one as a duplicate.