This topic was marked solved by its starter, on June 15th, 2013, 08:10 PM
« Can't Post Thoughts on IE10 on Win8
Undefinded function: sendemail()
Database error -- search for members »

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670
A confident man keeps quiet.whereas a frightened man keeps talking, hiding his fear.

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
Re: Undefinded function: sendemail()
« Reply #2, on June 15th, 2013, 10:59 AM »
Oops, should be sendmail. That line only handles e-mails when there's a notification bump, instant e-mail for new notification works, if you've set your notifications on periodical then I fixed that in my previous patch.
The way it's meant to be

Nao

  • Dadman with a boy
  • Posts: 16,082
Re: Undefinded function: sendemail()
« Reply #3, on June 15th, 2013, 06:17 PM »
Lulz... That explains a lot.

And nope, nothing in the error log... Which can only mean one thing: Wedge never even gets to that point, meaning there's another error left...

Dragooon is the champion of super-cool features that are totally, completely, utterly untested... :niark:

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
Re: Undefinded function: sendemail()
« Reply #4, on June 15th, 2013, 06:19 PM »
Quote from Nao on June 15th, 2013, 06:17 PM
Lulz... That explains a lot.

And nope, nothing in the error log... Which can only mean one thing: Wedge never even gets to that point, meaning there's another error left...

Dragooon is the champion of super-cool features that are totally, completely, utterly untested... :niark:
It doesn't get to the point because generally nothing takes it to the point. That line handles notification emails for which there already exists a notification, for example if someone liked your post and another person likes the same post and you had an old notification, that old notification will be bumped. Here currently nothing is bumping old notification really so it doesn't email anything. Although AFAIK that line *is* untested so fat chance it might actually be broken :P.

Nao

  • Dadman with a boy
  • Posts: 16,082

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670
Re: Undefinded function: sendemail()
« Reply #6, on June 15th, 2013, 08:10 PM »
Quote
And nope, nothing in the error log... Which can only mean one thing: Wedge never even gets to that point, meaning there's another error left...
Crap.
Quote
It doesn't get to the point because generally nothing takes it to the point. That line handles notification emails for which there already exists a notification, for example if someone liked your post and another person likes the same post and you had an old notification, that old notification will be bumped.
That's right.
Quote
Here currently nothing is bumping old notification really so it doesn't email anything.
Wrong..

It works . I got an email for a second like, and not the first.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
When we unite against a common enemy that attacks our ethos, it nurtures group solidarity. Trolls are sensational, yes, but we keep everyone honest. | Game Memorial

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670
Re: Undefinded function: sendemail()
« Reply #10, on June 16th, 2013, 12:49 AM »
Then no emaill. This is only if you set to instant. Set likes to notify instantly and wait until one of your posts get liked twice or more.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670
Re: Undefinded function: sendemail()
« Reply #14, on June 16th, 2013, 01:26 AM »
All periodical notifications are disabled here :P


   public function handleMultiple(Notification $notification, array &$data, array &$emaiL_data)
   {
      return false;
   }

That means "disable periodical notifications for this notification".
« Can't Post Thoughts on IE10 on Win8
Database error -- search for members »