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

David_Rothstein’s picture

This 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.

robloach’s picture

Status: Needs review » Needs work

Moved the files... http://drupalcode.org/project/field_permissions.git/commitdiff/382bca1

Let's get the merge in a patch...

robloach’s picture

Status: Needs work » Fixed

Yup!

Status: Fixed » Closed (fixed)

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