This is a port of popular live stats component for Joomla CMS:
JoomlaWatch allows you to watch your joomla visitors live stats and bots in real-time from the administration back-end. Specially their IP addresses, countries they come from, geographical location on a map, which pages they are viewing, their browser and operating system, it creates daily and all-time stats from these information plus unique, pageload and total hits statistics. Furthermore, you can block harmful IP addresses, see blocked attempts stats, evaluate the trend charts, and create goals based on many parameters. In the front-end, it displays the top countries, user and visit information for certain periods of time.
- translations: brazilian portuguese, dutch, french, german, greek, russian, slovak, slovenian, spanish, swedish, czech, danish, lithuanian, polish italian, turkish, latvian, ukrainian, bulgarian, albanian, bengali, chinese-simplified, chinese-traditional, estonian, indonesian, arabic, japanese, hebrew, croatian, hungarian, macedonian, ukrainian, vietnamese
Sandbox link:
http://drupal.org/sandbox/matto3c/1430416
Drupal core version: 7.x
Demo server:
http://www.codegravitydemo.com/drupal-free/
username: demo
password: demo
Security certificate located in: joomlawatch/administrator/components/com_joomlawatch/security_certificate.pdf
Comments
Comment #1
svenaas commentedAs a fellow project applicant I thought I'd pull my weight and take a look. First of all, it would save reviewers time if you'd add your Git clone URL, which is http://git.drupal.org/sandbox/matto3c/1430416.git. I tried to run your code through ventral's online PAReview code checker, but that seems to be having trouble at the moment (or perhaps just with your code)--this should be tried at another time as well--so I moved on to the code itself, where I spotted a couple of the same minor problems I had to fix with my own module: you're using the master branch instead of a version-specific branch (here's how to fix it) and you lack the expected README file.
Comment #2
matto3c commentedThank you for fast reply,
I moved it to a separate branch as instructed in the link.
I also created required README.txt and INSTALL.txt
Comment #3
matto3c commented-- double post removed ---
Comment #4
patrickd commentedhi matto3c,
unfortunately I have to please you not to run your code on ventral.org/pareview. As your code is still mostly joomla style (what is very different to drupal) the drupal code sniffer makes my 8-core/60gb ram server having trouble^^
I ran it manually but this is not the full review, I had to abort it after 1 min as it would run for hours otherwise.
See http://ventral.org/pareview/httpgitdrupalorgsandboxmatto3c1430416git
regards
Comment #5
svenaas commentedThat might have been me, patrickd; I tried it several times the other night and it stalled at 33%. I hope it didn't cause you any trouble, and I apologize if it did.
Comment #6
patrickd commentedno need to apologize, how should you know
just email me if everything hangs up next time ;-)
so back to topic
Comment #7
matto3c commentedHi guys,
thanks for update.. so it means that I have to put prefix to all my functions?
But this is mostly object-oriented code, so there should be no collisions whatsoever.
I completely separated and freed up my code from dependency on Joomla only (whoo-hoo),
as you can see I have one interface and each CMS implements these methods it's own way.
Looking forward for pointing me in the right direction what should I change in order to be compliant with Drupal.
Thank you and have a nice weekend.
Matej Koval
Comment #8
matto3c commentedBtw. I also updated the live demo link: http://www.codegravitydemo.com/drupal/
username: demo / password: demo
Comment #9
matto3c commentedno update on this?
Comment #10
patrickd commentedSorry for the delay, too many applications, too few reviewers.
You can get a review quicker by helping out in the issue queue your self.
Have a look at http://drupal.org/node/1410826
Comment #11
matto3c commentedI renamed the project:
http://drupal.org/node/1452328
Comment #12
patrickd commentedComment #13
avpaderno