The secure CMS.

Welcome to Geeklog, Anonymous Saturday, June 03 2023 @ 11:27 pm EDT

Geeklog Forums

submitloginrequired issue

 New Topic  Post Reply
 05/10/10 07:32am (Read 2,704 times)  
Status: offline

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1569
Location:la rochelle, France
Geeklog 1.7.0 is out and in the list of changes we can reed "The template file submit/submitloginrequired.thtml is no longer needed and has been removed."
but I know some plugins using it.

So if you use last mediagallery (1.5.1) plugin or the new maps beta plugin and the professional theme, please add the submit/submitloginrequired.thtml until the upgrade of these plugins.

::Ben
I'm available to customise your themes or plugins for your Geeklog CMS
 Quote
 05/10/10 07:41am  
Status: offline

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1569
Location:la rochelle, France
Here is some code from the calendar plugin to update plugins using the submitloginrequired.thtml

Before
Text Formatted Code
if (empty ($_USER['username']) &&
    (($_CONF['loginrequired'] == 1) || ($_CA_CONF['calendarloginrequired'] == 1))) {
    $display .= COM_siteHeader('');
    $display .= COM_startBlock ($LANG_LOGIN[1], '',
                                COM_getBlockTemplate ('_msg_block', 'header'));
    $login = new Template($_CONF['path_layout'] . 'submit');
    $login->set_file (array ('login'=>'submitloginrequired.thtml'));
    $login->set_var ( 'xhtml', XHTML );
    $login->set_var ('login_message', $LANG_LOGIN[2]);
    $login->set_var ('site_url', $_CONF['site_url']);
    $login->set_var ('site_admin_url', $_CONF['site_admin_url']);
    $login->set_var ('layout_url', $_CONF['layout_url']);
    $login->set_var ('lang_login', $LANG_LOGIN[3]);
    $login->set_var ('lang_newuser', $LANG_LOGIN[4]);
    $login->parse ('output', 'login');
    $display .= $login->finish ($login->get_var('output'));
    $display .= COM_endBlock (COM_getBlockTemplate ('_msg_block', 'footer'));
    $display .= COM_siteFooter();
    COM_output($display);
    exit;
}


After
Text Formatted Code
if (COM_isAnonUser() &&
    (($_CONF['loginrequired'] == 1) || ($_CA_CONF['calendarloginrequired'] == 1))) {
    $display .= COM_siteHeader('menu', $LANG_CAL_1[41]);
    $display .= SEC_loginRequiredForm();
    $display .= COM_siteFooter();
    COM_output($display);
    exit;
}


::Ben
I'm available to customise your themes or plugins for your Geeklog CMS
 Quote
 05/10/10 04:50pm  
Status: offline

Dirk

Site Admin
Admin
Registered: 01/12/02
Posts: 13073
Location:Stuttgart, Germany
Plugins that would like to be compatible with earlier versions could also check for the existence of that file and the SEC_loginRequiredForm() function and use whatever's there.

But yeah, I hadn't considered that this could be a problem when removing the file. My use case was the Forum plugin, which doesn't use that file ...

bye, Dirk
 Quote
 New Topic  Post Reply
All times are EDT. The time is now 11:27 pm.
  • Normal Topic
  • Sticky Topic
  • Locked Topic
  • New Post
  • Sticky Topic W/ New Post
  • Locked Topic W/ New Post
  •  View Anonymous Posts
  •  Able to post
  •  Filtered HTML Allowed
  •  Censored Content