31
anderssk
Re: First comments on 2.4
  • 2009/8/12 17:19

  • anderssk

  • Quite a regular

  • Posts: 335

  • Since: 2006/3/21


This is not enough
le="color: #000000"><?php "/(^|[^]_a-z0-9-="'/:.])([a-z0-9-_.]+?)@([{$valid_chars}]+[{$end_chars}])/ei"


The local part of an email adresse (the characters before the @-sign) can contain;
* Uppercase and lowercase English letters (a-z, A-Z)
* Digits 0 through 9
* Characters ! # $ % & ' * + - / = ? ^ _ ` { | } ~
* Character . (dot, period, full stop) provided that it is not the first or last character, and provided also that it does not appear two or more times consecutively.

Same as in bug 1988039

32
ghia
Re: First comments on 2.4
  • 2009/8/12 20:37

  • ghia

  • Community Support Member

  • Posts: 4953

  • Since: 2008/7/3 1


For the www (second) pattern, maybe this:
/(^|[^]_a-z0-9-=\"'\/:\.])www\.((([a-zA-Z0-9\-]*\.){1,}){1}([a-zA-Z]{2,6}){1})((\/([a-zA-Z0-9\-\._\?\,\'\/\\+&%\$#\=~])*)*)/
The replacement must be
"'\\1<a href=\"http://www.\\2\\6\" title=\"www.\\2\\6\" rel=\"external\">'.MyTextSanitizer::truncate( 'www.\\2\\6' ).'</a>'"

For the email (fourth) pattern, maybe this:
"/(^|[^]_a-z0-9-=\"'\/:\.])([-_a-z0-9\'+*$^&%=~!?{}]++(?:\.[-_a-z0-9\'+*$^&%=~!?{}]+)*+)@((?:(?![-.])[-a-z0-9.]+(?<![-.])\.[a-z]{2,6}|\d{1,3}(?:\.\d{1,3}){3})(?::\d++)?)/ei"

The bug were you reference to is at another place (checking for registering). If it uses also a regexp, then this part could be used:
[-_a-z0-9\'+*$^&%=~!?{}]++(?:\.[-_a-z0-9\'+*$^&%=~!?{}]+)*+@(?:(?![-.])[-a-z0-9.]+(?<![-.])\.[a-z]{2,6}|\d{1,3}(?:\.\d{1,3}){3})(?::\d++)?

33
DuGris
Re: First comments on 2.4
  • 2009/8/13 12:09

  • DuGris

  • Core Developer

  • Posts: 67

  • Since: 2004/3/14


on install, this line return sql error and configuration item is not inserted

install/include/makedata.php
le="color: #000000"><?php $dbm->insert('config', " VALUES (111, 0, 1, 'systemkey', '_MD_AM_SYSTEMKEY', '".buildXoopsSystemKey()."', '_MD_AM_SYSTEMKEYDSC', 'textbox', 'text', 99999)");

34
Mamba
Re: First comments on 2.4
  • 2009/8/13 12:26

  • Mamba

  • Moderator

  • Posts: 11521

  • Since: 2004/4/23


What environment?

35
trabis
Re: First comments on 2.4
  • 2009/8/13 13:53

  • trabis

  • Core Developer

  • Posts: 2269

  • Since: 2006/9/1 1


Hi ghia, the email pattern is working fine, thanks! the www pattern is producing good link but is echoing 'myTextSanitizer...etc between > and PS: I added /ei and seems to work fine!

36
DuGris
Re: First comments on 2.4
  • 2009/8/13 13:56

  • DuGris

  • Core Developer

  • Posts: 67

  • Since: 2004/3/14


Quote:

Mamba wrote:
What environment?


le="color: #000000"><?php XOOPS Version - XOOPS 2.4.0 Beta PHP Version - 5.2.8 MySQL Version - 5.0.51b-community-nt Server API Version - apache2handler OS Version - WINNT safe_mode - Off register_globals - Off magic_quotes_gpc - Off allow_url_fopen - On fsockopen - On allow_call_time_pass_reference - Off post_max_size - 8M max_input_time - 60 output_buffering - 4096 max_execution_time - 6000 memory_limit - 128M file_uploads - On upload_max_filesize - 2M

37
ghia
Re: First comments on 2.4
  • 2009/8/13 15:00

  • ghia

  • Community Support Member

  • Posts: 4953

  • Since: 2008/7/3 1


Quote:
the www pattern is producing good link but is echoing 'myTextSanitizer...etc
Use the existing replacement and change all http://www.\\2.\\3 to http://www.\\2\\6

Quote:
I added /ei and seems to work fine!

Sorry, I added only the strings/content.

38
djynnius
Re: First comments on 2.4
  • 2009/8/14 20:03

  • djynnius

  • Just popping in

  • Posts: 59

  • Since: 2006/10/21


Great seeing XOOPS 2.4! especially with the multi database capability. but i got a few questions.
1. how do you install on an alternate database (that is other than mysql) since i did not see the option in the dropdown menu while installing.

2. whats the XOOPS system key about?

39
DuGris
Re: First comments on 2.4
  • 2009/8/16 19:30

  • DuGris

  • Core Developer

  • Posts: 67

  • Since: 2004/3/14


Fixed on svn (rev 3451)

40
trabis
Re: First comments on 2.4
  • 2009/8/16 20:15

  • trabis

  • Core Developer

  • Posts: 2269

  • Since: 2006/9/1 1


Quote:
trabis wrote: Hi ghia, the email pattern is working fine, thanks! the www pattern is producing good link but is echoing 'myTextSanitizer...etc between > and PS: I added /ei and seems to work fine!
Added to SVN, thanks Ghia!

Who's Online

270 user(s) are online (205 user(s) are browsing Support Forums)


Members: 0


Guests: 270


more...

Donat-O-Meter

Stats
Goal: $15.00
Due Date: Jul 31
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $15.00
Make donations with PayPal!

Latest GitHub Commits