There is a discussion of what should be in D8: http://drupal.org/node/1419652
One request is: Add watchdog interface for javascript code to use: http://drupal.org/node/727278
jDog, http://drupal.org/sandbox/peter/1414906, adds both the watchdog interface and an email interface.

The code is in use on D7 sites. There appears to be little difference for D6 or D8. Watchdog has not changed much. The jQuery Ajax function was added before D6 started using jQuery. The main change appears to be the admin settings page. I do not have a D6 or D8 site for testing the admin page in D6 or D7.

I also have no experience setting up module tests on drupal.org. Assistance in that area would produce a very quick spread to D8.

I added a readme.txt file because some pages recommend the file and the pareview page complained there is no README.txt file.

The code is created in Netbeans on Linux but pareview complained the code contains email line endings, crnl, instead of nl. I used jEdit to change the line endings.

I changed the master to a branch using Netbeans Git. pareview appears to see several branches. Another todo.

There are a couple of things to do before it is finished for the current sites. Some more values will be passed to the Javascript file. The site URL is next on the list. It will then handle everything required by the current sites.

The message delimiter will be user selectable one day. The watchdog substitution will be emulated in the email code. Both of these are required for the next site.

At that point the version one code will be finished for all the current sites for the watchdog messaging and basic site email.

A further change will let the Javascript supply the email address, something not required for current sites but it makes a lot of sense for future flexibility.

D7 loads the .module file for hook_install but not hook_uninstall so the code will load the .module file for _uninstall.

Reviews of other projects

After years of experience searching for modules, I reviewed for what they say about their modules. Could I decide to use the module based on the description?

http://drupal.org/node/1167022#comment-5596872
http://drupal.org/node/1421554#comment-5596760
http://drupal.org/node/1269194#comment-5598582
http://drupal.org/node/1387212#comment-5598628

Comments

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
commented
Status: Active » Needs work

Please read https://drupal.org/node/1011698 and update this issue accordingly.

The response time for a review is now approaching 4 weeks. Get a review bonus and we will come back to your application sooner.

peterx’s picture

peterx commented

I am seriously considering giving up on this.
* The Ventral site has a stupid login where I have to type the captcha 12 times before it works.
* The report complains about line endings created by the Netbeans project code that I have no control over and never sent to drupal.org.
* The Ventral report lists two branches. I created only one, according to Netbeans.
* Most of the Drupal.org documentation refers to DOS box commands. Why are we still stuck in the 1950s?
* If I push on one computer then pull on another, I end up with code that never existed on any of the computers and I have to retype everything. CVS was never this bad.

Sorry for the rant. I am used to professional version control software. Git is a real embarrassment to everyone in the software industry. I tried to use Git for several projects. Git failed on every one. When Git has a master copy and nothing else, I can sometimes push and pull code from one computer to another without damage. Using branches is Russian roulette.

patrickd’s picture

patrickd commented
Category: feature » task

note that most of the coding style issues pointed by automated tools are suggestions!
the application process is mostly about general code quality, security issues and licensing.
Anyway if you don't like to use the online tool, you can still install pareview localy, there are several tutorials about this.

peterx’s picture

peterx commented

I started using SmartGit again and and found a way to make it workable. SmartGit appeared to be randomly deleting files. It is actually randomly not displaying files and there are switches you can set to stop it being so stupid. The switches do have a use but the defaults of off, with no warning, is painful. Plus there is no documentation about the switches and no tutorial on how to use it.

peterx’s picture

peterx commented
Status: Needs work » Closed (duplicate)

I am closing this one in favour of Cel, http://drupal.org/node/1528036, because I have no active projects depending on jDog and will not be able to work on it for a while. A number of developers are already using Cel, are asking for a drupal.org version, and I am actively working with it for the next few weeks, so battling with PAReview is practical.

peterx’s picture

peterx commented
Status: Closed (duplicate) » Needs review

jDog will return with tuning of the email section.

peterx’s picture

peterx commented
Status: Needs review » Closed (duplicate)
peterx’s picture

peterx commented
Issue summary: View changes

Added reviews of other modules.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Unleash the Drupal watchdog in jQuery » [D7] Unleash the Drupal watchdog in jQuery
Related issues: +#1528036: [D7] Cel