Is there any way of adding a nother filter on this please ?

[joad]

Hi all, Happy Easter and isn't it great to have a brilliant excuse to eat all those chocs!! Anyways, can anyone help on this? Have been trying for a few weeks to get this right, need to do it so that it's a pre-requisite to do 100 posts to post an email address and 50 posts to post a website address.

if ($submit || $preview || $refresh)
{

if ($user->data['user_posts'] < 100)
   {
      if (preg_match('/(http|www)/i', request_var('message', '', true)))
      {
         $error[] = $user->lang['NO_URLS'];
      }
  	 if (preg_match('/(@)/i', request_var('message', '', true)))
  	  {
         $error[] = $user->lang['NO_EMAILS'];
      }


   }

Any help much appreciated (I can't afford to pull out any more hair!!) 🙂

Joad

[laserlight]

For starters, you should indent the code properly, e.g.,

if ($submit || $preview || $refresh)
{
    if ($user->data['user_posts'] < 100)
    {
        if (preg_match('/(http|www)/i', request_var('message', '', true)))
        {
            $error[] = $user->lang['NO_URLS'];
        }

    if (preg_match('/(@)/i', request_var('message', '', true)))
    {
        $error[] = $user->lang['NO_EMAILS'];
    }
}

As for the logic: I would expect something like this:

if number of posts < 100 and email address in text:
    add error message to list of error messages
if number of posts < 50 and website address in text:
    add error message to list of error messages

[joad]

Many thanks for the help - much appreciated - I'll try that code and report back.

I didn't get the logic part? Is that to be coded somewhere?

[joad]

if ($submit || $preview || $refresh)
{
    if ($user->data['user_posts'] < 100)
    {
        if (preg_match('/(http|www)/i', request_var('message', '', true)))
        {
            $error[] = $user->lang['NO_URLS'];
        }

    if (preg_match('/(@)/i', request_var('message', '', true)))
    {
        $error[] = $user->lang['NO_EMAILS'];
    }
} 

Is it my imagination, but is there a { too many ?

[joad]

Also, how can it be coded so that it's a pre-requisite to do 100 posts to post an email address and 50 posts to post a website address.

[laserlight]

joad wrote:

I didn't get the logic part? Is that to be coded somewhere?

Yes, it is pseudocode that describes one way to solve your problem.

joad wrote:

Is it my imagination, but is there a { too many ?

It is not your imagination: in the code you posted, the first opening brace does not have a closing brace.

joad wrote:

Also, how can it be coded so that it's a pre-requisite to do 100 posts to post an email address and 50 posts to post a website address.

It depends on what exactly you mean. The pseudocode I showed already does that, as in the user would need at least 100 posts to be allowed to post and email address, and 50 posts in order to be allowed to post a website address.

[jgetner]

if number of posts < 100 and email address in text:
add error message to list of error messages
if number of posts < 50 and website address in text:
add error message to list of error messages

been using a little to much python i might suspect with : instead of {

[laserlight]

jgetner wrote:

been using a little to much python i might suspect with : instead of {

Good observation. I prefer indentation, not braces, to indicate scope for pseudocode, hence I decided to base my style on Python. However, my pseudocode style is irrelevant to this discussion.

[jgetner]

sorry don't mean to hi jack the this discussion was just a little phunny to me as some habbits are die hards...

[joad]

Many thanks for the help on this - all now solved 🙂