Okay, so the Incubator just really needs an image gallery. For basic functionality I'm thinking:

Gallery display: http://drupal.org/project/galleryformatter
File uploads: http://drupal.org/project/plupload

To integrate this into the incubator:

  1. download required modules and libraries, adding each one to bahai_incubator.make
  2. create a new module like incubator_galleries
  3. make sure dependencies are noted in .info file
  4. configure module dependencies as needed, and code all steps in new module's .install file (generally variable_set statements)
  5. create new content types, fields, and views, and test until satisfied
  6. track new blocks enabled and code in new module's .install file as per other incubator modules
  7. export content types using devel module and this code
  8. copy and paste field definitions into new module's .install file as per other incubator modules
  9. handle the new content type in incubator_galleries.install as per other incubator modules
  10. create a patch as per the instructions and post it back here
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dnotes’s picture

This is better now:

Formatter:
http://drupal.org/project/galleryformatter

Multiple uploader:
http://drupal.org/project/plupload

dnotes’s picture

Make sure to consult the http://drupal.org/packaging-whitelist when adding the external library to the bahai_incubator.make file.

dnotes’s picture

Assigned: Unassigned » djones1844
dnotes’s picture

Issue tags: +Milestone 1
dnotes’s picture

Issue summary: View changes

update for current phase of incubator work

dnotes’s picture

Issue summary: View changes

Further clarification on adding stuff to incubator

djones1844’s picture

Component: Modules » Install Profile
Status: Active » Needs review
FileSize
5.42 KB

This should add the gallery modules to the required install as well as creating a content type and roles/permissions for a photographer.

dnotes’s picture

Status: Needs review » Needs work

Awesome! The first collaboration on the incubator!

Okay, here are some things that need work:

  1. The code currently in the _install() hook should go in the _enable() hook.
  2. When creating the content type, use _incubator_create_nodetype(). See one example in incubator_localities.install. This will ensure that no errors happen if the content type already exists, as well as reminding you to specify the pathauto prefix, default comment settings, and whether or not to show the submitted info.
  3. I'd prefer that the field be named "gallery_images" instead of "picture", to further distinguish it from the main_image field.
  4. Field and instance definitions should go in a separate function, e.g. _incubator_fields_field_gallery_images. See incubator_localities.install for an example. Also note that some parts of the field definition should be removed, most notably all the settings for the 'storage' key and the entirety of the 'bundles' key.
  5. Use the _incubator_attach_field function to attach the field to the content type.
  6. Use the _incubator_create_role function to make the photographer role.
  7. When the module is disabled, we must clean up the data with hook_disable() and hook_uninstall(). Check out incubator_events.install for an example.
dnotes’s picture

Status: Needs work » Fixed
FileSize
1.92 KB

This is the patch that was eventually used.

dnotes’s picture

Oops wrong patch.

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

Anonymous’s picture

Issue summary: View changes

added patch instructions