Re: C-JAY ReviewsExcellent
Re: C-JAY Reviewsa quick fix: produces (at least for me) an include error. I also changed it to the single-quote style because it's || that much faster 
Find (file: myReviews/admin/index.php ~Line 38)
include("../../mainfile.php"); include('../../../mainfile.php'); Re: C-JAY Reviewsanother fix
Fix: Doesn't work with register_globals = off
Find (File: myReviews/admin/index.php ~Line 1575)
switch ($op) { $op = (!empty($_POST['op'])) ? $_POST['op'] : $_GET['op'];
switch ($op) { Re: C-JAY ReviewsAnother fix
Fix: Inclues error
Find (File: myReviews/include/dlformat.php ~Line 3)
include("../cache/config.php"); include(XOOPS_ROOT_PATH.'modules/myReviews/cache/config.php'); 
Re: C-JAY ReviewsForlon did state it was a beta 
Re: C-JAY Reviewsyeah, that's why i thought it could be an alpha version. in any case, it looks good :). defiantly not bashing the author. if it seemed that way, my apologies.
Re: C-JAY Reviews@zer0fill
Thank you very much for you fixes. I will inlcude them today. You maybe do have a better knowledge in PHP then I do.
I just wondering why you call it fixes. I do not have any errors on my site and on my testing machine.
I definetly does not yet support Register Globals = off.
And some of you tips are just another way to do it and not really errors in my eyes.
Cheers,
Don't get me wrong. I am glad for you help!
Re: C-JAY Reviewsi tested it on my site with PHP Debug enabled. if you haven't tried it yet, try enabling PHP Debug. the only "errors" you should be worried about are warnings and errors; notices are nothing to be concerned about
HTH
Re: C-JAY ReviewsTHis version is beta. Therefore there are error in PHP debug.
Nevertheless the module works an I think does not have to many errors as you mentioned some threads before.
Or isn't it working on you site? What enviroment to you have then?
Cheers,
Re: C-JAY ReviewsI have done a major redesign of the code to get it working with globals off, use the XOOPS install framework and strip all the language references into the language folders for multi language support. I still need to do smarty integration, but will do that later. Next thing on the list is comment blocks and notification. If there is anyone willing to do some alpha testing please contact me. I am looking at doing different review categories, but will do that after stabilization.
Grotmis
Riaan
http://www.craftsonline.co.za
| Stats | |
| Goal: | $100.00 |
| Due Date: | Nov 30 |
| Gross Amount: | $0.00 |
| Net Balance: | $0.00 |
| Left to go: | $100.00 |
 |
|