Instead of having all of these different directories, we should just have everything in the module directory, prefixing the filename with "field_permissions."...
- includes/admin.inc
- field_permissions.admin.inc
- includes/field_access.inc
- field_permissions.access.inc (maybe even merge this file into field_permissions.module)
- css/field_permissions.admin.css
- field_permissions.admin.css
- css/field_permissions.field.settings.css
- field_permissions.settings.css (merge into field_permissions.admin.css?)
- images/field-permissions-status-off.png
- images/field_permissions.status-off.png
- images/field-permissions-status-on.png
- images/field_permissions.status-on.png
- js/field_permissions.field.settings.js
- field_permissions.admin.js
Thoughts?
Comments
Comment #1
David_Rothstein commentedThis plan makes a lot of sense to me. And I agree with the two merging suggestions also. The module isn't really big enough for all those split out files to have that much benefit.
Comment #2
robloachMoved the files... http://drupalcode.org/project/field_permissions.git/commitdiff/382bca1
Let's get the merge in a patch...
Comment #3
robloachYup!