Posted by MarcoR on December 15, 2007 at 10:27am
| Project: | Birthdays |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | MarcoR |
| Status: | closed (fixed) |
Issue Summary
Especially comments on functions need to be improved. See http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/api/README....
But also some less important naming conventions should be adopted (i.e. $output for themed html as return value).
I can take that in some says. Because this will change many places in the code, it's better to do that when no heavy development is going on then. So please submit your latest changes to cvs.
Comments
#1
There is no heavy development going on at the moment, nor are there plans for that in the near future, due to sickness.
Feel free to do what you suggested, because it's one of those to-dos I never can find the time for.
Thanks for your input!
#2
I was very happy that birthday module has improved so much compared from the last time I saw it. Reallly very good! I fixed the following:
- Function comments according to core drupal functions and api module
<?php/**
* @desc headline
*/
?>
to
<?php/**
* headline
*/
?>
- $str to $output
- $u to $user
- $values to $form_values (that should have fixed an issue where a form could not be set back to defaults)
- set some function comments except where I'm not sure what type the arguments would be
- other minor indentation issues
As far as I tried it's working as perfect as before. I think this is a good start for further develeopment.
#3
I forgot: Also made a JavaScript improvement: FILTER Button is not shown if JavaScript is enabled. ;)
#4
It would be nice if some people could test this patch. Because so many lines have changed, going ahead with any other birthdays issue only makes sense when this patch has been committed. Thanks! :)
#5
Now that I'm getting well again, I'll look into your patch and commit it.
#6
#7
Automatically closed -- issue fixed for two weeks with no activity.