The vB Geek

Go Back   The vB Geek > vB Geek Products > Geek Gazette

Geek Gazette GAZ is a community newsletter management system.

Advertisement
  Learn how to remove ads

Reply
 
Thread Tools Display Modes
  #1  
Old 03-26-2007, 06:04 PM
Milad Milad is offline
Member
 
Join Date: Nov 2005
Posts: 58
Geek Article and Review System License Holder Geek Gazette License Holder 
Default GAZ 1.0.0 beta 1 feedback and suggestions from Milad

Installation process:
- As of vB 3.6 you can add the cron job in the XML product file, I didn't like the fact that I had to setup a cron job manually while that can be done via XML file.

readme.html file:
correct the path: http://www.yoursite.com/forums/geek/..._send_cron.php to http://www.yoursite.com/forums/geek/...n/gaz_send.php


Settings:
- I found my personal email in Reply address setting, and this must be the default one of the forum $vbulletin->options['webmasteremail']
- Style and user information (Style, Userid and User Name): Here it's better to use the own setting for each user, and it's better to send the newsletter from the default email $vbulletin->options['webmasteremail'] not from the admin membership.

Newsletters:
- It's better to determine the scheduled time for each newsletter while adding or editing it not via the cron job which have to run to decide if it's the time to generate an issue or not. (I figured out later, this can be set in generators section)
- timing could be daily, weekly, biweekly and monthly .. etc.

Issues:
- Management here is good.

Generator:
- Management here is good.

Found errors:
- In Generators section, I chosen Preview last HTML issue, then new window popped up with this url http://www.syrianmeds.net/forum/.php?id=1&prev=1
- Also the link for Plain Text preview is http://www.syrianmeds.net/forum/?settings[scriptname].php?id=1&txt=1&prev=1
- The previous problems disappeared in newly generated issues, I think it's default data problem.
- In modules section, I receive this warning: Warning: Missing argument 5 for run_mod() in /geek/gaz/includes/gaz_class_core.php on line 226
- In Plain Text preview, there are HTML code.
- In sending section for an issue I had this error Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22
- While you add a module to module set, if you set an order, it won't be set, you have to right it in order field and save again.

Sent HTML issue:
- It must have the HTML direction is wrong, it's ltr while my forum is rtl
- I choose not to show replies and views but their fields are still exist as empty fields.
- some parts don't obey right-to-left standards.

Notes:
- templates have to be grouped in the admincp
Reply With Quote
  #2  
Old 03-26-2007, 09:01 PM
KirbyDE KirbyDE is offline
Member
 
Join Date: Sep 2005
Posts: 31
Geek Article and Review System License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

Quote:
Originally Posted by Milad View Post
Settings:
- Style and user information (Style, Userid and User Name): Here it's better to use the own setting for each user, and it's better to send the newsletter from the default email $vbulletin->options['webmasteremail'] not from the admin membership.
The sender eMail should indeed be configurable.
Also, it might be a good idea to offer 'Use User Style' as an option, although this will cause more processing.

Quote:
Found errors:
- In Generators section, I chosen Preview last HTML issue, then new window popped up with this url http://www.syrianmeds.net/forum/.php?id=1&prev=1
- Also the link for Plain Text preview is http://www.syrianmeds.net/forum/?settings[scriptname].php?id=1&txt=1&prev=1
This has already been reported: http://www.thevbgeek.com/tracker/ind...md=view&id=329

Quote:
- In modules section, I receive this warning: Warning: Missing argument 5 for run_mod() in /geek/gaz/includes/gaz_class_core.php on line 226
This has already been reported: http://www.thevbgeek.com/tracker/ind...md=view&id=333
Reply With Quote
  #3  
Old 03-26-2007, 11:04 PM
The Geek's Avatar
The Geek The Geek is offline
Senior Member
 
Join Date: Aug 2005
Posts: 6,717
Geek Article and Review System License Holder GeekMart License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

Thanks for taking the time to write this up Milad, it really helps!

Quote:
Originally Posted by Milad View Post
Installation process:
- As of vB 3.6 you can add the cron job in the XML product file, I didn't like the fact that I had to setup a cron job manually while that can be done via XML file.
There are 2 cron jobs: 1 for the generators, 1 to send out mail. THe generator cron job is done via the install, however I dont link the send cron to vB's system because there is FAR too much processing to do for vB's schedule task system.

There is no other way around this (other than sending the emails out manually which I have made pretty automated too).

Thanks

Quote:
Settings:
- I found my personal email in Reply address setting, and this must be the default one of the forum $vbulletin->options['webmasteremail']

- Style and user information (Style, Userid and User Name): Here it's better to use the own setting for each user, and it's better to send the newsletter from the default email $vbulletin->options['webmasteremail'] not from the admin membership.
Good point - however here is the real crux... if the style is per the users settings... it could potentially be a WHOLE lot of additional processing. I suppose the way to do it would be to pre-parse each potential style (escaping the user specific variables like $user). That would work a little bit better.

Quote:
Newsletters:
- It's better to determine the scheduled time for each newsletter while adding or editing it not via the cron job which have to run to decide if it's the time to generate an issue or not. (I figured out later, this can be set in generators section)
Yes, the vB cron job for running the generator uses the generators time and frequency setting to create new issues. The send cron sends whatever outstanding issues it can find.

Quote:
Found errors:
- In Generators section, I chosen Preview last HTML issue, then new window popped up with this url http://www.syrianmeds.net/forum/.php?id=1&prev=1
- Also the link for Plain Text preview is http://www.syrianmeds.net/forum/?settings[scriptname].php?id=1&txt=1&prev=1
- The previous problems disappeared in newly generated issues, I think it's default data problem.
- In modules section, I receive this warning: Warning: Missing argument 5 for run_mod() in /geek/gaz/includes/gaz_class_core.php on line 226
- In Plain Text preview, there are HTML code.
- In sending section for an issue I had this error Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22
- While you add a module to module set, if you set an order, it won't be set, you have to right it in order field and save again.
Thanks!

Quote:
Sent HTML issue:
- It must have the HTML direction is wrong, it's ltr while my forum is rtl
I had to add a forced setting to make it LTR because the cron was defaulting to RTL (because no user was actually loading it, there was no defualt language direction in vB). I just didnt put the option in to toggle RTL on of off

Quote:
- I choose not to show replies and views but their fields are still exist as empty fields.
Try a different template style (i.e. simple threadlist instead of threadlist)

Thanks again for your feedback. It helps loads
Reply With Quote
  #4  
Old 03-27-2007, 02:54 AM
Milad Milad is offline
Member
 
Join Date: Nov 2005
Posts: 58
Geek Article and Review System License Holder Geek Gazette License Holder 
Default

Quote:
There are 2 cron jobs: 1 for the generators, 1 to send out mail. THe generator cron job is done via the install, however I dont link the send cron to vB's system because there is FAR too much processing to do for vB's schedule task system.

There is no other way around this (other than sending the emails out manually which I have made pretty automated too).
I faced something like that before, I had to run a maintenance script once per hour, I wasn't able to do that via CPanel, because wget is denied, I made a plug-in (via cron_script_cleanup hook) that can be run via vB's crons, it was like this:

PHP Code:
$getthisfile file('http://PATH TO /maintenance.php');
unset(
$getthisfile); 
You can add this in small file to handle the mission.

Quote:
because no user was actually loading it, there was no default language direction in vB
But there is a default language that has a direction, you can use it. I think this is better than toggle option.

Quote:
Try a different template style (i.e. simple threadlist instead of threadlist)
Yes I tried this, it's better.

Can you add an option to use the actual header and footer and navbar in the newsletters?

I'm waiting for beta 2

Thanks
Reply With Quote
  #5  
Old 03-27-2007, 08:15 AM
The Geek's Avatar
The Geek The Geek is offline
Senior Member
 
Join Date: Aug 2005
Posts: 6,717
Geek Article and Review System License Holder GeekMart License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

Hey Milad,

The problem isnt in finding a place with vB's cron job - it is the way that vBs heduled task system works. vb's heduled task is not a real cron - it runs off the back end of another user. There is a web site that you can schedule tasks on for free. Basically you register the URL and time with the site and it calls the script at that time. That would be the best way forward. Maybe Kirb has another idea.

It was much more complex than that. I too assumed that would be the case, but it was a challenge to load up a mock user, language and style via a cron and via a normal page. The interim solution is a setting.

I wont be adding the option for the regular header and footer simply because it will consist of sending javascript files and functions too. I suspect there could be serious problems with that - maybe Kirby can comment?
Reply With Quote
  #6  
Old 03-27-2007, 09:24 AM
KirbyDE KirbyDE is offline
Member
 
Join Date: Sep 2005
Posts: 31
Geek Article and Review System License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

I see no real problem with running a heavy processing script through vB Cron Jobs. They are triggered via a 1 Pixel Image, so they don't really affect the browsing user triggering execution.

Things to keep in mind though: It might be necesary ton increase max_execution time and ignore user abort.
Also, if processing takes > 1 minute (which is probably the case) it might be necessary to build the next execution time before the end of script execution, so it does not run into race conditions.
Reply With Quote
  #7  
Old 03-27-2007, 01:59 PM
Milad Milad is offline
Member
 
Join Date: Nov 2005
Posts: 58
Geek Article and Review System License Holder Geek Gazette License Holder 
Default

I assure I understand how vB's crons work, via that 1 pixel image which is cron.php?cronid=XX. me too, don't see a real problem with running a heavy processing script through vB Cron Jobs.

Best regards
Reply With Quote
  #8  
Old 03-27-2007, 02:02 PM
Milad Milad is offline
Member
 
Join Date: Nov 2005
Posts: 58
Geek Article and Review System License Holder Geek Gazette License Holder 
Default

Quote:
I wont be adding the option for the regular header and footer simply because it will consist of sending javascript files and functions too. I suspect there could be serious problems with that - maybe Kirby can comment?
So I have to drop those files and functions from my header and footer, then to replace the GAZ templates with them.

What is about automated dropping process?
Reply With Quote
  #9  
Old 03-27-2007, 02:45 PM
The Geek's Avatar
The Geek The Geek is offline
Senior Member
 
Join Date: Aug 2005
Posts: 6,717
Geek Article and Review System License Holder GeekMart License Holder Geek Auto-Linker Pro License Holder Geek Advertising Banner License Holder Geek Gazette License Holder 
Default

The GAZ header and navbar use most of the default header. The navbar is slightly different as it doesn't contain any javascript menus, nor irrelevant items. Customising it is pretty straight forward and simple.

As for the cron job - I have never seen anything apart from caution from vb staff about its cron jobs being used for massive tasks - hence my hesitation from utilising vBs scheduled task system for this purpose. TBH it is easier to set it up as a cron job (it originally was) so I assume that it will get folded back in.
Reply With Quote
  #10  
Old 03-28-2007, 03:15 PM
Milad Milad is offline
Member
 
Join Date: Nov 2005
Posts: 58
Geek Article and Review System License Holder Geek Gazette License Holder 
Default

I browse to the file gaz_send.php and I get this repeated error:

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22

Warning: Missing argument 1 for gaz_class_cron() in /geek/gaz/includes/gaz_class_cron.php on line 22
Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

Similar Threads
Thread Thread Starter Forum Replies Last Post
GAZ Private Beta 3 Release Discussion The Geek Geek Gazette 12 04-21-2007 01:22 AM
GAZ Private Beta 2 Release Discussion The Geek Geek Gazette 5 04-11-2007 12:40 AM
feedback kidmercury Geek Article and Review System 0 11-01-2005 04:01 AM


All times are GMT. The time now is 10:47 AM.


Powered by vBulletin® Version 3.8.5
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.