Advertisement:

Author Topic: $smcFunc defined twice as global  (Read 653 times)

Offline doug_ips

  • Jr. Member
  • **
  • Posts: 139
    • generalcartalk.com
$smcFunc defined twice as global
« on: March 04, 2018, 08:42:25 AM »
Hello,

The $smcFunc var is being defined twice as global in the Register.php file.

Code: [Select]
// Begin the registration process.
function Register($reg_errors = array())
{
global $txt, $boarddir, $context, $settings, $modSettings, $user_info;
global $language, $scripturl, $smcFunc, $sourcedir, $smcFunc, $cur_profile;
It's fast, it's furious, it's generalcartalk.com

Check out my mods

Offline albertlast

  • Development Contributor
  • Jr. Member
  • *
  • Posts: 276
Re: $smcFunc defined twice as global
« Reply #1 on: March 04, 2018, 09:19:17 AM »
In smf 2.1 this issue didn't exists,
maybe got already fixed.

Offline d3vcho(void)

  • Localizer
  • SMF Hero
  • *
  • Posts: 2,518
  • Gender: Male
  • Be prepared for hell
    • d3vcho on GitHub
    • @SMF_ES on Twitter
Re: $smcFunc defined twice as global
« Reply #2 on: March 04, 2018, 09:27:39 AM »
Yes, it was fixed by Arantor a long time ago for SMF 2.1 but it's still there for 2.0.x. I'll pass it to the developers.

Thank you.

Offline Shambles

  • SMF Hero
  • ******
  • Posts: 4,723
  • Gender: Male
    • i30 Owners Club
Re: $smcFunc defined twice as global
« Reply #3 on: March 04, 2018, 09:34:15 AM »
I wouldn't class this as a bug since it doesn't cause incorrect or unexpected results, or for Register.php to behave in unintended ways.

Just sayin'

Offline d3vcho(void)

  • Localizer
  • SMF Hero
  • *
  • Posts: 2,518
  • Gender: Male
  • Be prepared for hell
    • d3vcho on GitHub
    • @SMF_ES on Twitter
Re: $smcFunc defined twice as global
« Reply #4 on: March 04, 2018, 09:37:14 AM »
Nope it's not a bug, it's just repeated code that should be removed. But it's better to keep it here so we don't forget about it.

Offline doug_ips

  • Jr. Member
  • **
  • Posts: 139
    • generalcartalk.com
Re: $smcFunc defined twice as global
« Reply #5 on: March 04, 2018, 01:54:30 PM »
Indeed, it is not really a bug per se. But I did not know where else to post it. And yes, it appears that in the 2.1. version it has been fixed.
It's fast, it's furious, it's generalcartalk.com

Check out my mods