Jump to content


Photo
- - - - -

Few fixes


  • Please log in to reply
1 reply to this topic

#1 dargre

dargre

    Senior Member

  • Members
  • PipPipPipPipPip
  • 988 posts

Posted 14 March 2016 - 03:19 PM

Sometimes this can be serious issue....

 

For all ajax php file system doesn't load config file, doesn't load debug file with display_errors status.

So every ajax file may show php notices, warnings, depreciated etc. This can break ajax response, nothing will show on website for ajax actions. Most often this may concern related videos, though not only.

 

Fix is simple, just edit file "ajax.php" and above line:

require 'include/config.paths.php';

enter

ini_set('display_errors', 0);

Another bug found in ajax "related_videos.php" concerns max total of related videos.

File video.php sets it to 32, but related changes it to 80. Should be same.

Line:

$total          = ( $total > 80 ) ? 80 : $total;

should be:

$total          = ( $total > 32 ) ? 32 : $total;

Big issue is also for those who upgraded from 3.0 or 3.1 to 3.2 or 3.3

New URL for each category is duplicated. Old URL like  "videos?c=2" is still working and indexed by search engines.

This also concerns AVS demo website.

Of course this can be fixed using proper redirection, but it's not 1 or 2 lines of code to have it properly, additionally different for each of 3 types of categories. I don't use AVS script daily, but one of clients found it harmful for his website an asked to fix and so I did.

 

 


  • Damien and devil360 like this

VIDEO.JS - best HTML5 player in the world for AVS script

NUEVOPLAYER 8 - best flash+html5 player for video scripts
Mods, custom code by NuevoLab (Search Suggest, Multiple image upload, Seo mods,and much more)


#2 devil360

devil360

    Member

  • Members
  • PipPip
  • 24 posts

Posted 14 March 2016 - 04:57 PM

Thanks Dargre.