[D7] Domain Feeds

Hey,

Have taken a look at the code part of this module, haven't tested the functionality as of yet.

• looks like it could use som dependencies in the info file,
• remove the $id$ part - it's deprecated

• ($data['domain_id'] == 0) ? $key = -1 : $key = $data['domain_id'];
  

  could (and should as it's more clear whats going on) be rewritten to

  $key = ($data['domain_id'] == 0) ? -1 : $data['domain_id'];
  

• "else if" on line 46 should be moved to a new line
• Missing newline at end of file.

except for the above mentions the code looks fine

forgot to change status

Just because i'm picky of nature there is still a little code-formatting issue, but nothing that should prevent you from having this module upgraded to full project status.

For future work remember to run your code through the coder module, it helps correct a lot of the minor issues like whitespace at the end of a line.

Congratulations, Jim.M!  You can now promote sandbox projects to full status ones.  When you would like further review for any project please do ask at http://groups.drupal.org/peer-review/requests

Further cleanup issues can be filed on your project issue queue.  Thank you for your contribution and i look forward to seeing its refinement and development and your continued involvement in Drupal!

_{benjamin, agaric}

Jim.M - both when you edit the project near the project shortname and as a link on the project page.

Apparently not... wonder if i accidentally vetted some random passerby... ;-) Should be all set now.