The secure CMS.

Welcome to Geeklog, Anonymous Saturday, March 14 2026 @ 03:32 pm EDT

Geeklog Forums

submitloginrequired issue

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

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1581
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
 Quote
 05/10/10 07:41am  
Status: offline

::Ben

Forum User
Full Member
Registered: 01/14/05
Posts: 1581
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
 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 03:32 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