I am using the Shorten URL in a text using Bit.ly PHP class by Muhammad Arfeen and I would like to replace his code with Philippe Leybaert’s code. The original PHP code is below:

preg_match_all(’(((f|ht){1}(tp://|tps://))[-a-zA-Z0-9@:%_+.~#?&//=]+)’, $text, $hyperlinksArray);

Here is the same code with Philippe Laybaert’s regular expression:

preg_match_all(’\b(https|http)://([a-zA-Z-:@.0-9]+)(/((\([-;:@&=a-zA-Z0-9$_.+!*\',]*?\))|[-;:@&=?a-zA-Z0-9$_.+!*\',]|%\d\d)+)?(?< ![,.;])', $text, $hyperlinksArray);

Since he did not enclose his code in quotes, I used single quotes like in the original code and escaped the two single quotes in the regex with backslashes.

My problem is every time this code is parsed it returns “Warning: preg_match_all() [function.preg-match-all]: Delimiter must not be alphanumeric or backslash in [file] on [line]” and no replacements are made. Is this an issue with the regex itself, escaping, or something with PHP? I am using PHP 5. An example of $text would be “Check out this cool Wikipedia page: http://en.wikipedia.org/wiki/Textile_(disambiguation)” and the last string is defined as `$hyperlinksArray = array();`.

The problem with the original regex is that it does not work with parentheses and it includes periods, commas, semicolons, and close parenthesis at the end of a URL in the URL itself, which, in all cases except semicolons, is NOT what Twitter does. Since I am shortening URLs with Bit.ly, using the original code on my example will produce a Bit.ly short URL for the long URL http://en.wikipedia.org/wiki/Textile_ instead of http://en.wikipedia.org/wiki/Textile_(disambiguation) as it should. Philippe’s code would solve these problems and several others if I could get it to work.

I am using the Shorten URL in a text using Bit.ly PHP class by Muhammad Arfeen and I would like to replace this code with yours:

preg_match_all(’(((f|ht){1}(tp://|tps://))[-a-zA-Z0-9@:%_+.~#?&//=]+)’, $text, $hyperlinksArray);

I tried copying and pasting your code between the single quotes and escaping the two single quotes in your final regex:

preg_match_all(’\b(https|http)://([a-zA-Z-:@.0-9]+)(/((\([-;:@&=a-zA-Z0-9$_.+!*\',]*?\))|[-;:@&=?a-zA-Z0-9$_.+!*\',]|%\d\d)+)?(?< ![,.;])', $text, $hyperlinksArray);

However, every time this code is parsed it returns “Warning: preg_match_all() [function.preg-match-all]: Delimiter must not be alphanumeric or backslash in [file] on [line].”

I am not an expert on regular expressions so I don’t know how to fix this. Do you have any advice? I’ve been looking for hours for regex code like yours and was really hoping it would work.

Thank you so much for helping me resolve this issue.

Your Plugin is wonderful and easy to use I will recommend it to all my friends.

Once again thank you so much for your help.

Cesar

It appears you are using 1&1 hosting which used PHP 4 by default until recently. Try adding this line to the top of your .htaccess file at /htdocs/wordpress-2.9.2/wordpress/.htaccess :

AddType x-mapp-php5 .php

I disabled all the plugins and it still doesn’t work. The Error occurs after I click on the publish to publish a new or update a posting.

Thank you

I also forgot to mention that once I Click Test Twitter OAuth it never displays “Authentication succeeded” it only says testing… but nothing seams to be happening.

Thank you very much for your help.

I have been using you plugin with some month now and let me tell you it’s a great one. I just recently upgraded to your new version 1.7.1. I have followed all the direction and haven’t being able to make it work. I am getting this error

Parse error: syntax error, unexpected T_STRING, expecting T_OLD_FUNCTION or T_FUNCTION or T_VAR or ‘}’ in /homepages/41/d276329632/htdocs/wordpress-2.9.2/wordpress/wp-content/plugins/tweet-this/lib/twitteroauth.php on line 19

Can you please guide me on the best way to fixing this error. I am running wordpress 3.0.1.

Best Regards,

Cesar

~Dave