Module:eCard

mam885

Joined: 2013-07-03
Posts: 8
Posted: Tue, 2013-07-09 12:15

Hello community

i'm using the e-card-module for G3, but it doesn't work. When I try to send a card, the forms (receiver, message, ...) appear, but when I use the "send" button, an error message appears:

Quote:
Platform information

Host name: tux113
Operating system: Linux 2.6.36.3-default
Apache: Unknown
PHP: 5.3.8
MySQL: 5.5.28
Server load: 1.66 1.43 1.28
Graphics toolkit: imagemagick

Gallery stats

Version: 3.0.4 (Ricochet)
Albums: 41
Photos: 3326

Exception [ 0 ]: @todo SEND_MAIL_FAILED

MODPATH/gallery/libraries/Sendmail.php[ 90 ]
85 // The docs say headers should be separated by \r\n, but occasionaly that doesn't work and you 86 // need to use a single \n. This can be set in config/sendmail.php 87 $headers = implode($this->header_separator, $headers); 88 $message = wordwrap($this->message, $this->line_length, "\n"); 89 if (!$this->mail($to, $this->subject, $message, $headers)) { 90 throw new Exception("@todo SEND_MAIL_FAILED"); 91 } 92 return $this; 93 } 94 95 public function mail($to, $subject, $message, $headers) {

MODPATH/ecard/controllers/ecard.php[ 120 ] » Sendmail_Core->send()

MODPATH/ecard/controllers/ecard.php[ 80 ] » Ecard_Controller::_notify( arguments )

{PHP internal call} » Ecard_Controller->send( arguments )

SYSPATH/core/Kohana.php[ 331 ] » ReflectionMethod->invokeArgs( arguments )

{PHP internal call} » Kohana_Core::instance( arguments )

SYSPATH/core/Event.php[ 208 ] » call_user_func_array( arguments )

APPPATH/Bootstrap.php[ 67 ] » Event_Core::run( arguments )

DOCROOT/index.php[ 113 ] » require( arguments )

Environment

i noticed following line:
85 // The docs say headers should be separated by \r\n, but occasionaly that doesn't work and you 86 // need to use a single \n. This can be set in config/sendmail.php

but there's no config-folder in root-folder of the gallery. In folder [g3-root]/modules/gallery/libraries/ is a file named "sendmail.php". But modifying this as it was recommended in the error message, doesn't help. This is the sendmail.php-file, the part what's bold is the one I removed for test.

Quote:
<?php defined("SYSPATH") or die("No direct script access.");
/**
* Gallery - a web based photo album viewer and editor
* Copyright (C) 2000-2012 Bharat Mediratta
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* the Free Software Foundation; either version 2 of the License, or (at
* your option) any later version.
*
* This program is distributed in the hope that it will be useful, but
* WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* General Public License for more details.
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA 02110-1301, USA.
*/
class Sendmail_Core {
protected $to;
protected $subject;
protected $message;
protected $headers;
protected $line_length = 70;
protected $header_separator = "\r\n";

/**
* Return an instance of Sendmail
* @chainable
*/
static function factory() {
return new Sendmail();
}

public function __construct() {
$this->headers = array();
$this->from(module::get_var("gallery", "email_from", ""));
$this->reply_to(module::get_var("gallery", "email_reply_to", ""));
$this->line_length(module::get_var("gallery", "email_line_length", 70));
$separator = module::get_var("gallery", "email_header_separator", null);
$this->header_separator(empty($separator) ? "\n" : unserialize($separator));
}

public function __get($key) {
return null;
}

public function __call($key, $value) {
switch ($key) {
case "to":
$this->to = is_array($value[0]) ? $value[0] : array($value[0]);
break;
case "header":
if (count($value) != 2) {
Kohana_Log::add("error", wordwrap("Invalid header parameters\n" . Kohana::debug($value)));
throw new Exception("@todo INVALID_HEADER_PARAMETERS");
}
$this->headers[$value[0]] = $value[1];
break;
case "from":
$this->headers["From"] = $value[0];
break;
case "reply_to":
$this->headers["Reply-To"] = $value[0];
break;
default:
$this->$key = $value[0];
}
return $this;
}

public function send() {
if (empty($this->to)) {
Kohana_Log::add("error", wordwrap("Sending mail failed:\nNo to address specified"));
throw new Exception("@todo TO_IS_REQUIRED_FOR_MAIL");
}
$to = implode(", ", $this->to);
$headers = array();
foreach ($this->headers as $key => $value) {
$key = ucfirst($key);
$headers[] = "$key: $value";
}

// The docs say headers should be separated by \r\n, but occasionaly that doesn't work and you
// need to use a single \n. This can be set in config/sendmail.php
$headers = implode($this->header_separator, $headers);
$message = wordwrap($this->message, $this->line_length, "\n");
if (!$this->mail($to, $this->subject, $message, $headers)) {
throw new Exception("@todo SEND_MAIL_FAILED");
}
return $this;
}

public function mail($to, $subject, $message, $headers) {
return mail($to, $subject, $message, $headers);
}
}

Thank you very much for your help in advance.
Michael