Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

detect non-projects and fail gracefully #1

Open
NewAmsterdamIdeas opened this issue Feb 24, 2012 · 1 comment
Open

detect non-projects and fail gracefully #1

NewAmsterdamIdeas opened this issue Feb 24, 2012 · 1 comment

Comments

@NewAmsterdamIdeas
Copy link

This module came in very handy for me in compiling data about a website for a security review. I used it in combination with the node_import module to generate a listing of stats for all of the contrib modules in use on the website.

I did notice some bugginess when the module tries to import information about projects that don't have a corresponding project page on Drupal.org. The buggy imports occurred because I compiled my list of modules via a database query to the system table, and some of the modules listed there are submodules of projects rather than actual projects in their own right. JQuery barfed when it tried to retrieve the stats for those modules. I think it would be a good idea to have some failover code that does a better job of handling situations when someone has tried to reference a project page on Drupal that doesn't exist. Wish I had time to write the patch myself, but I thought I should at least post the suggestion.

Thanks again for a great module.

@greggles
Copy link
Owner

Thanks, @sheldonrampton :)

I gave this a new title, hopefully it matches what you think should happen. It would be great if you could share the script you used for importing. I'm sure lots of other users of the module will want to do that as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants