Re: [drupal-devel] Introduction and Questions
I've read the contributors guide, which states I should "and then notify the contributions manager to request a review of your work before posting". I can't see anything that tells me how to go about this, so I am assuming the method is to post to the list? Should I check into contributions before review to have it somewhere to be reviewed, or post it in some other form somewhere? Or does "before posting" mean before filling in the form to list under modules?
I have attached the module in a RAR, I hope this won't cause too much offence, it is less than 1kb.
Yeah the "contributions manager" thing should probably be changed to something clearer. You can commit anything you think is reasonable into the contrib repository (and of course within the rules), but it won't show up on Drupal.org until you create a project and have it approved by one of the people with the mojo to do this. Your module seems okay, granted it's also very short ;). I have two comments: - You search for the string "livejournal" in the server name, which would include more than the real livejournal.com. Using a regexp anchored to the end is probably a better idea (e.g. '/livejournal\.com$/i'). Granted, this issue will probably never pop up, but it's still nice to be exact. - There are a couple of things that don't fit the Drupal code style (a couple of spaces and a brace). This doesn't matter for the contributions repository, as they are not required there, but if you plan on submitting core patches later, it might be a good idea to start with good habits. If you want a review next time, it's probably best to just commit it to the repository and ask on this list. The people here are almost all developers, so keep that in mind. If you want to reach general Drupal users, try the forums instead. As far as attachments go, if they are small (like yours) then it's acceptable, though I would suggest using .tar.gz or .zip instead of .rar as they are more widespread. Steven Wittens
Your module seems okay, granted it's also very short ;)
Now I've got to go and buy a sports car or something to compensate... But seriously, unless I was to implement the entire livejournal xml rpc api in the module, there isn't really anything to add.
- You search for the string "livejournal" in the server name, which would include more than the real livejournal.com. Using a regexp anchored to the end is probably a better idea
Well, I have to admit to personal reasons for not using a regex and just using a stristr. I want users to be able to login to my site as just user@livejournal. I was thinking about the blogger authentication module, there are other big sites that support the blogger api. Would it be "best" in Drupal convention terms to: 1) Provide a map of $server -> $url in an array (for example) within the module to handle the different paths? (i.e. modblog.com uses /?remote as the path) Or 2) Provide a different authentication module for each service with a different url?
- There are a couple of things that don't fit the Drupal code style
Oops. I can't run the code-style.pl, no perl. I've corrected the bracing on the if's and the spaces in the xmlrpc constructors. Was there anything else? Thanks for the feedback, Mike
If you want a review next time, it's probably best to just commit it to the
Committed, and project request submitted to drupal.org. I've branched the module to DRUPAL-4-5, which I think is correct for getting it available to 4.5 package list? See, that's why my module is so small. I'm learning the process rather than coding anything important ;-) Mike
participants (2)
-
Michael Jervis -
Steven Wittens