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
"/(^|[^]_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'.MyTextSanitizer::truncate( 'www.\\2\\6' ).''"

For the email (fourth) pattern, maybe this:
"/(^|[^]_a-z0-9-=\"'\/:\.])([-_a-z0-9\'+*$^&%=~!?{}]++(?:\.[-_a-z0-9\'+*$^&%=~!?{}]+)*+)@((?:(?![-.])[-a-z0-9.]+(?
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.]+(?

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
$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: 11409

  • 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?


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!

Login

Who's Online

306 user(s) are online (231 user(s) are browsing Support Forums)


Members: 0


Guests: 306


more...

Donat-O-Meter

Stats
Goal: $100.00
Due Date: Nov 30
Gross Amount: $0.00
Net Balance: $0.00
Left to go: $100.00
Make donations with PayPal!

Latest GitHub Commits