CVS edit link for pglatz

I have been building modules for the last three years and feel I have reached the level of expertise to make valuable contributions back to the community. I also would like to become co-maintainer of some other projects I use that have not been translated to 6.x and 7.x yet.

Here are a couple of short modules for developers I find handy and would like to submit:
http://www.electronovelty.com/content/watchspy
http://www.electronovelty.com/content/nodedump

a few others:
http://www.electronovelty.com/page/contributed-modules

I try to be a good Drupal citizen, and regret I have not contributed much code back to the community. I follow the guidelines and use code review and validators. My experience the last three years at consumersearch.com have made me aware of the issues developers of large sites face, and the importance of doing things the Drupal way.

CommentFileSizeAuthor
#1 nodedump.zip20.35 KBpglatz
#1 watchspy.zip31.27 KBpglatz
#5 watchspy.zip14.56 KBpglatz
#5 nodedump.zip8.62 KBpglatz

Comments

pglatz’s picture

Status: Needs work » Postponed (maintainer needs more info)
Issue tags: -Module review
StatusFileSize
new31.27 KB
new20.35 KB

Please use the updated versions of these from #5, thanks.

pglatz’s picture

Status: Postponed (maintainer needs more info) » Needs review
avpaderno’s picture

Status: Needs review » Needs work
Issue tags: +Module review

Hello, and thanks for applying for a CVS account. I am adding the review tags, and some volunteers will review your code, pointing out what needs to be changed.

As per http://drupal.org/cvs-application/requirements, the motivation message should be expanded to contain more details about the features of the proposed module, and it should include also a comparison with the existing solutions.
We then review a module per applicant; let us know which one you want to be reviewed.

pglatz’s picture

Thanks. Please review the watchspy module.

Motivation: watchspy is a primary diagnostic tool for Drupal developers, and
contains very useful information. The user interface, however, is pretty bare
bones. It does allow filtering by type and severity, but not by date range. The
output is presented as a summary; to read the details of a message requires
opening a new browser tab or window. I find the clicking to read full message
texts can be a lot of work.

Watchspy eliminates much of the clicking by displaying full message details in
an expanded hidden div when the mouse is above the summary. You can stiull click
on the message to see the full details in another tab or window, but most of the
time the simple mouseover is all you need.

It has two additional features that can be useful:
1) Filter the date range of messages.

2) The ability to purge classes of message types from the log, handy if you
want to start a new test and look for specific messages.

It also has a settings page, where you can specify the number of rows per page
and control the appearance of the summary screen.

I am not aware of other modules that allow an expansion of watchdog message
details with a mouseover.

TODO:
1) Add filter by servity

pglatz’s picture

StatusFileSize
new8.62 KB
new14.56 KB

I just uploaded corrected zips of watchspy and node dump (duh, I had included .svn info with them). I also cleaned up some of the logic and styling for watchspy; please grab this one instead of the original version I posted.

pglatz’s picture

Status: Postponed (maintainer needs more info) » Needs work
Issue tags: +Module review

I have been asked to co-maintain Search Log and will need CVS access. Please see http://drupal.org/node/788746

avpaderno’s picture

Status: Needs work » Fixed

Thank you for your contribution! I am going to update your account.
These are some recommended readings to help with excellent maintainership:

You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.

I thank all the dedicated reviewers as well.

Status: Fixed » Closed (fixed)
Issue tags: -Module review

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

avpaderno’s picture

Component: Miscellaneous » co-maintainer application
Assigned: Unassigned » avpaderno
Issue summary: View changes