mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #36847
[Bug 1608682] [NEW] Remove obsolete print_auth_frame() code
Public bug reported:
Code added in 2008 for ajaxlogin around file upload was mostly removed
in 2009 but a few remnants were left behind.
See comment in patchset #9 for https://reviews.mahara.org/#/c/5542
Regarding the comment about the deleted function 'show_login_form()'
being used in htdocs/lib/htmloutput.php
It is used because htdocs/auth/lib.php calls the php function print_auth_frame() inside the
if (defined('IFRAME')) { }
But it doesn't look like we define IFRAME anywhere.
Ah but we used to in 2008 ->
f7fe047518b3a21b9d12323da7d5d338a33e6d8b
But got rid of the stuff relating to it in 2009 ->
fdf71aa0db535f39815a53d0677af674ce2a8d80
So this is leftover code and can be removed
** Affects: mahara
Importance: Low
Assignee: Robert Lyon (robertl-9)
Status: Invalid
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1608682
Title:
Remove obsolete print_auth_frame() code
Status in Mahara:
Invalid
Bug description:
Code added in 2008 for ajaxlogin around file upload was mostly removed
in 2009 but a few remnants were left behind.
See comment in patchset #9 for https://reviews.mahara.org/#/c/5542
Regarding the comment about the deleted function 'show_login_form()'
being used in htdocs/lib/htmloutput.php
It is used because htdocs/auth/lib.php calls the php function print_auth_frame() inside the
if (defined('IFRAME')) { }
But it doesn't look like we define IFRAME anywhere.
Ah but we used to in 2008 ->
f7fe047518b3a21b9d12323da7d5d338a33e6d8b
But got rid of the stuff relating to it in 2009 ->
fdf71aa0db535f39815a53d0677af674ce2a8d80
So this is leftover code and can be removed
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1608682/+subscriptions
Follow ups