Get XOOPS XOOPSXOOPS FAQFAQ ForumsForums NewsNews ThemesThemes ModulesModules
New Posts New Topics All Posts All Forums Index General Modules Themes Development International XOOPS.org

Search


Advanced Search

Donat-O-Meter

Make donations with PayPal!
Stats
Goal: $100.00
Due Date: Jun 30
Gross Amount: $25.00
Net Balance: $23.57
Left to go: $76.43
Donations
Anonymous ($15)Jun-14
Anonymous ($10)Jun-10

Learn XOOPS Core

Click to see original Image in a new window
OOP with XOOPS

XOOPS API
XOOPS Blue Move XOOPS Giftshop XOOPS Blue Move
Share


Local Support

Advertisement

XOOPS Code hosted on SourceForge

Cumulus Tag Cloud

2 2.5 2.6 3.0 87 2013 Abuse admin Amazon AntiHarvesting AntiMalUser AntiSpam API bags Beats billige black Blocks blue Bootstrap Captcha cell Christmas chronolabs content Conversion demo docek download Dresses editor evden eve facebook floor free herre Honeypot Human IP IPInfoDB jQuery kantor lamps Language log logger Lucire mobile module modules Monster mulberry MyAlbum-p newbb news online PageRank pandora Password Permissions pink Plugin portal preloader Prevention profile project propose Protector publisher Rights rmcommon Room sale security Server site Smarty SOAP Songlist Spam stem Studio tag tags tdmcreate template Theme themes TinyMCE User userlog website Whitepaper Wishcraft XIPS xoops Xortify ZendFramework
more...

New Users

Registering user

# 136115

EdEstes

Welcome to XOOPS!

Forum Index

Board index » XOOPS Modules Support forums » Module troubleshooting » Liaise 1.27 and Xoops 2.3.0 RC2




Bottom   Previous Topic   Next Topic  Register To Post

(1) 2 3 4 5 »


#1 Posted on: 2008/9/7 13:32 Liaise 1.27 and Xoops 2.3.0 RC2
Hi!

I'm testing XOOPS 2.3.0 RC2 and has just installed Liaise 1.27 and when I press the send button I recive a new page with this text:

Errors
Failed opening template file.

If I go to the adminpage I see this text:
Version of XOOPS does not meet the system requirement. Liaise may not work properly.

Anyone knows how to fix this?
____________
Best Regards
Ahlis

Top

AHLIS
Not too shy to talk
Not too shy to talk
Joined:
2004/12/12 9:00
From Sweden
Group:
Registered Users
Posts: 172
(Show More) (Show Less)


#2 Posted on: 2008/9/7 15:06 Re: Liaise 1.27 and Xoops 2.3.0 RC2
I'm interested in this too..... good post

What happens when you try with php debugging turned on, i.e. are there any error messages?

Cheers

Top

John V
Cardiff - UK
JAVesey
Moderator
Moderator
Joined:
2006/10/19 16:01
From Cardiff - UK
Group:
Registered Users
Community Coordinator (temporary)
Posts: 2296
(Show More) (Show Less)


#3 Posted on: 2008/9/7 15:19 Re: Liaise 1.27 and Xoops 2.3.0 RC2
If you look in liaise/admin/footer.php you will see that liaise is expecting to find XOOPS 2.0.9 (at least in liase 1.23)

Top

Xoops User Utilities | Xoops Search Engine | vil3/trabisdementia
trabis
Core Developer
Core Developer
Joined:
2006/9/1 13:10
From Portugal
Group:
Webmaster
Registered Users
Posts: 2198
(Show More) (Show Less)


#4 Posted on: 2008/9/7 15:33 Re: Liaise 1.27 and Xoops 2.3.0 RC2
Quote:
trabis wrote:

If you look in liaise/admin/footer.php you will see that liaise is expecting to find XOOPS 2.0.9 (at least in liase 1.23)


Interesting..... in Liaise v1.26, line 42 of footer.php says:

$version_check = preg_match('/2.0.[9|10|11|12|13]/', XOOPS_VERSION);


Interestingly, it works fine with XOOPS 2.0.18.2.

Presumably, changing the code to something like that below would sort out the Admin message?:

$version_check = preg_match('/2.3.[0|1|2|3|4]/', XOOPS_VERSION);


What about the template issue, though?

Top

John V
Cardiff - UK
JAVesey
Moderator
Moderator
Joined:
2006/10/19 16:01
From Cardiff - UK
Group:
Registered Users
Community Coordinator (temporary)
Posts: 2296
(Show More) (Show Less)


#5 Posted on: 2008/9/7 16:09 Re: Liaise 1.27 and Xoops 2.3.0 RC2
For what I can see there is a bug in XOOPS 2.3 and you need to do some changes in liaise also.

in liaise include/form_execute (around line 184 for liaise 1.26)
you will find:
if( is_dir(LIAISE_ROOT_PATH."language/".$xoopsConfig['language']."/mail_template") ){
    $template_dir = LIAISE_ROOT_PATH."language/".$xoopsConfig['language']."/mail_template";
}else{
    $template_dir = LIAISE_ROOT_PATH."language/english/mail_template";
}

$xoopsMailer =& getMailer();
$xoopsMailer->setTemplateDir($template_dir);


relpace by this:
$xoopsMailer =& xoops_getMailer();
$xoopsMailer->setTemplateDir('liaise');


And thenyou need to fix the core bug in class/xoopsmailer.php line 180:
$path = XOOPS . "/modules/" . $path . "/languages/";

Replace by
$path = XOOPS_ROOT_PATH . "/modules/" . $path . "/language/";


Top

Xoops User Utilities | Xoops Search Engine | vil3/trabisdementia
trabis
Core Developer
Core Developer
Joined:
2006/9/1 13:10
From Portugal
Group:
Webmaster
Registered Users
Posts: 2198
(Show More) (Show Less)


#6 Posted on: 2008/9/7 16:35 Re: Liaise 1.27 and Xoops 2.3.0 RC2
Quote:
n liaise include/form_execute (around line 184 for liaise 1.26)
you will find:

if( is_dir(LIAISE_ROOT_PATH."language/".$xoopsConfig['language']."/mail_template") ){
    $template_dir = LIAISE_ROOT_PATH."language/".$xoopsConfig['language']."/mail_template";
}else{
    $template_dir = LIAISE_ROOT_PATH."language/english/mail_template";
}

$xoopsMailer =& getMailer();
$xoopsMailer->setTemplateDir($template_dir);
relpace by this:

$xoopsMailer =& xoops_getMailer();
$xoopsMailer->setTemplateDir('liaise');


I think you are breaking a standard here and compatibility with a lot of modules.
This problem has to be resolved entirely in phpmailer, so that it is ready to accept this kind of template specifications.

This is a code snippet from 2.2.6rc
// public
    function setTemplateDir($value)
    {
        if ( substr($value, -1, 1) != "/" ) {
            $value .= "/";
        }
        $this->templatedir = $value;
    }

    // public
    function setTemplate($value)
    {
        $this->template = $value;
    }


// public
    function send($debug = false)
    {
        global $xoopsConfig;
        if ( $this->body == "" && $this->template == "" ) {
            if ($debug) {
                $this->errors[] = _MAIL_MSGBODY;
            }
            return false;
        } elseif ( $this->template != "" ) {
            $path = ( $this->templatedir != "" ) ? $this->templatedir."".$this->template : (XOOPS_ROOT_PATH."/language/".$xoopsConfig['language']."/mail_template/".$this->template);
            if ( !($fd = @fopen($path, 'r')) ) {
                if ($debug) {
                    $this->errors[] = _MAIL_FAILOPTPL;
                }
                        return false;
                }
            $this->setBody(fread($fd, filesize($path)));
        }

        // for sending mail only


Top

The art of asking questions.
ghia
Community Support Member
Community Support Member
Joined:
2008/7/3 14:19
From Belgium
Group:
Registered Users
Posts: 4946
(Show More) (Show Less)


#7 Posted on: 2008/9/7 16:51 Re: Liaise 1.27 and Xoops 2.3.0 RC2
I agree. xoopsmailer needs to be reviewed to maintain compatibility. As it is, you do not even need to set a template dir and as you are forced to use mail_template.

// private
    function getTemplatePath()
    {
        if (!$path = $this->templatedir) {
            $path = XOOPS_ROOT_PATH . "/language/";
        } elseif ( false === strpos($path, '/') ) {
            $path = XOOPS . "/modules/" . $path . "/languages/";
        } elseif ( substr($path, -1, 1) != "/" ) {
            $path .= "/";
        }
        if (file_exists($path . $GLOBALS['xoopsConfig']['language'] . "/" . $this->template) ) {
            return $path . $GLOBALS['xoopsConfig']['language'] . "/mail_template/" . $this->template;
        } else {
            return $path . "english/mail_template/" . $this->template;
        }
    }


// public
    function send($debug = false)
    {
        global $xoopsConfig;
        if ( $this->body == "" && $this->template == "" ) {
            if ($debug) {
                $this->errors[] = _MAIL_MSGBODY;
            }
            return false;
        } elseif ( $this->template != "" ) {
            $path = $this->getTemplatePath();
            if ( !($fd = @fopen($path, 'r')) ) {
                if ($debug) {
                    $this->errors[] = _MAIL_FAILOPTPL;
                }
                return false;
            }
            $this->setBody(fread($fd, filesize($path)));
        }
...


Top

Xoops User Utilities | Xoops Search Engine | vil3/trabisdementia
trabis
Core Developer
Core Developer
Joined:
2006/9/1 13:10
From Portugal
Group:
Webmaster
Registered Users
Posts: 2198
(Show More) (Show Less)


#8 Posted on: 2008/9/7 16:57 Re: Liaise 1.27 and Xoops 2.3.0 RC2
I guess this would be a better fix for xoopsmailer.php

function getTemplatePath()
    {
        if (!$path = $this->templatedir) {
            $path = XOOPS_ROOT_PATH . "/language/";
        } elseif ( false === strpos($path, '/') ) {
            $path = XOOPS_ROOT_PATH . "/modules/" . $path . "/language/";
        } elseif ( substr($path, -1, 1) != "/" ) {
            $path .= "/";
        }
        
        if (file_exists($path . $GLOBALS['xoopsConfig']['language'] . "/mail_template/" . $this->template) ) {
            return $path . $GLOBALS['xoopsConfig']['language'] . "/mail_template/" . $this->template;
        } elseif (file_exists($path . "english/mail_template/" . $this->template) ) {
            return $path . "english/mail_template/" . $this->template;
        } elseif (file_exists($path . $this->template) ) {
            return $path . $this->template;
        } else {
            return false;
        }
    }


Top

Xoops User Utilities | Xoops Search Engine | vil3/trabisdementia
trabis
Core Developer
Core Developer
Joined:
2006/9/1 13:10
From Portugal
Group:
Webmaster
Registered Users
Posts: 2198
(Show More) (Show Less)


#9 Posted on: 2008/9/8 11:53 Re: Liaise 1.27 and Xoops 2.3.0 RC2
Quote:

trabis wrote:
I guess this would be a better fix for xoopsmailer.php

function getTemplatePath()
    {
        if (!$path = $this->templatedir) {
            $path = XOOPS_ROOT_PATH . "/language/";
        } elseif ( false === strpos($path, '/') ) {
            $path = XOOPS_ROOT_PATH . "/modules/" . $path . "/language/";
        } elseif ( substr($path, -1, 1) != "/" ) {
            $path .= "/";
        }
        
        if (file_exists($path . $GLOBALS['xoopsConfig']['language'] . "/mail_template/" . $this->template) ) {
            return $path . $GLOBALS['xoopsConfig']['language'] . "/mail_template/" . $this->template;
        } elseif (file_exists($path . "english/mail_template/" . $this->template) ) {
            return $path . "english/mail_template/" . $this->template;
        } elseif (file_exists($path . $this->template) ) {
            return $path . $this->template;
        } else {
            return false;
        }
    }


Thanks. The above code helped. :)
____________
Best Regards
Ahlis

Top

AHLIS
Not too shy to talk
Not too shy to talk
Joined:
2004/12/12 9:00
From Sweden
Group:
Registered Users
Posts: 172
(Show More) (Show Less)


#10 Posted on: 2008/9/8 12:08 Re: Liaise 1.27 and Xoops 2.3.0 RC2
Thread bookmarked

Top

John V
Cardiff - UK
JAVesey
Moderator
Moderator
Joined:
2006/10/19 16:01
From Cardiff - UK
Group:
Registered Users
Community Coordinator (temporary)
Posts: 2296
(Show More) (Show Less)



Board index » XOOPS Modules Support forums » Module troubleshooting » Liaise 1.27 and Xoops 2.3.0 RC2

Top   Previous Topic   Next Topic
(1) 2 3 4 5 »

  Register To Post


You can view topic.
You cannot start a new topic.
You cannot reply to posts.
You cannot edit your posts.
You cannot delete your posts.
You cannot add new polls.
You can vote in polls.
You cannot attach files to posts.
You can post without approval.
You cannot use topic type.
You cannot use HTML syntax.
You cannot use signature.

[Advanced Search]