Issue Details (XML | Word | Printable)

Key: MDL-15878
Type: Sub-task Sub-task
Status: Resolved Resolved
Resolution: Fixed
Priority: Minor Minor
Assignee: Petr Skoda (frankenstein)
Reporter: Eloy Lafuente (stronk7)
Votes: 0
Watchers: 0
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle
MDL-14589

Some DB implementation things

Created: 30/Jul/08 12:13 AM   Updated: 01/Aug/08 06:15 AM
Component/s: Files API
Affects Version/s: 2.0
Fix Version/s: 2.0

Participants: Eloy Lafuente (stronk7) and Petr Skoda (frankenstein)
Security Level: None
Resolved date: 01/Aug/08
Affected Branches: MOODLE_20_STABLE
Fixed Branches: MOODLE_20_STABLE


 Description  « Hide
Some things I found when looking to DB:
  • contenthash VARCHAR(40)... seems to be nullable. Is that correct?
  • itemid BIGINT(10) unsigned ... also is nullable. Aren't all files going to have itemid?
  • status BIGINT(10) ... stands for?
  • CREATE INDEX mdl_file_con_ix ON mdl_files (contenthash) , shouldn't this be unique too?
  • CREATE INDEX mdl_file_con2_ix ON mdl_files (contextid) , doesn't seem to be necessary as there is another index with "contextid" as first field.
  • CREATE INDEX mdl_fileclea_con_ix ON mdl_files_cleanup (contenthash) , if index above (by contexthash) is unique, this should be too.


 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Petr Skoda (frankenstein) added a comment - 30/Jul/08 04:55 AM
status is some error status - virus found, file missing, etc. 0 means file ok - not implemented yet

Petr Skoda (frankenstein) added a comment - 30/Jul/08 04:55 AM
patch updated, sending soon

Petr Skoda (frankenstein) added a comment - 01/Aug/08 06:15 AM
done, thanks