View Single Post
  #325  
Old 01-31-2006, 09:47 PM
trilljester's Avatar
trilljester trilljester is offline
 
Join Date: Jul 2005
Posts: 151
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by yayvb
Hmmm, I had installed this lastnight and woke to find my users complaining that when they clicked page 2 of a thread they would be brought to a completely different thread!! I checked and sure enough that's what was happening.

So even if I fix the fact that on the main page only the first threads are rewritten, apparently if there is more than 1 page of posts in a thread, there is a bug making it so you get transported to the wrong thread instead of page 2 of the right thread.

Anyone else have this problem? I didn't see this in here already.
It's happening to me too. I might have to turn this promising extension off. It appears to be way too unstable to be released in a non-test environment. I think the problem in the logic for gathering the URL to rewrite is flawed, and with no way to debug the output, it makes it that much harder to figure out where the problem is coming from.

EDIT: Here's a function I used a long time ago to strip characters that could cause problems when I was scanning a URL. It might be useful for this extension.

Code:
function make_url_safe($text) {
// Characters must be in ASCII and certain ones are not allowed.
 $text = html_entity_decode ($text);
 $text = str_replace(" / ","-",$text);
 $text = str_replace("/","-",$text);
 $text = str_replace("'","-",$text);
 $text = str_replace(" - ","-",$text);
 $text = str_replace("_","-",$text);
 $text = str_replace(" ","-",$text);
 $text = str_replace( "?", "ae", $text);
 $text = str_replace( "?", "oe", $text);
 $text = str_replace( "?", "ue", $text);
 $text = str_replace( "?", "Ae", $text);
 $text = str_replace( "?", "Oe", $text);
 $text = str_replace( "?", "Ue", $text);
 $text = str_replace( "?", "ss", $text);
 $text = str_replace( "&", "and", $text);
 $text = str_replace( "%", "Percent", $text);
 $text = ereg_replace("[^A-Za-z0-9-]", "", $text);
 $text = str_replace("----","-",$text);
 $text = str_replace("---","-",$text);
 $text = str_replace("--","-",$text);
return $text;
}
So pass the preg_replace'd URLs through this filter to help. I don't know, it's worth a shot.
Reply With Quote
 
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.01361 seconds
  • Memory Usage 1,771KB
  • Queries Executed 11 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD_SHOWPOST
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)bbcode_code
  • (1)bbcode_quote
  • (1)footer
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (6)option
  • (1)post_thanks_box
  • (1)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (1)post_thanks_postbit_info
  • (1)postbit
  • (1)postbit_onlinestatus
  • (1)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open 

Phrase Groups Available:
  • global
  • postbit
  • reputationlevel
  • showthread
Included Files:
  • ./showpost.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_postinfo_query
  • fetch_postinfo
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showpost_start
  • bbcode_fetch_tags
  • bbcode_create
  • postbit_factory
  • showpost_post
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • showpost_complete