Ticket #127 (closed Bug report: fixed)

Opened 12 years ago

Last modified 11 years ago

Remove dependency to default node

Reported by: kaouete@… Owned by: rob_janes
Priority: normal Milestone: WifiDog Auth Server 1.0
Component: Auth server, Other Version: Auth Server SVN
Keywords: Cc:

Description

Hi,

i just updated my svn this morning (r1025)

and i get this error when i try to login :

User Notice: Le noeud default n'a pu être trouvé dans la base de données ! in /var/www/wifidog/classes/Node.php on line 410

Change History

Changed 12 years ago by kaouete@…

I guess it is caused by [1018] for the file source:trunk/wifidog-auth/wifidog/login/index.php

Changed 12 years ago by lokkju

I talked to the reporter. Seems that this was changed from [1013] to [1018]/[1020]

Not quite sure how we should handle virtual logins - should we just require a "defualt" id, or should we use "null", as in [1013]?

Changed 12 years ago by benoitg

  • owner set to rob_janes
  • priority changed from high to normal
  • summary changed from the node default is not found to Remove dependency to default node
  • milestone set to WifiDog Auth Server 1.0

You probably deleted or renamed the default node. You must update your config.php to reflect that. That being said, there was indeed problem in [1018] where a hardcoded 'default' was used instead of DEFAULT_NODE_ID from config.php.

I didn't set the node back to null, as this would probably break part of Rob's changes for Smarty variable assignment. That being said, even we had just about managed to get completely rid of the dependance to the existence of a default node, so Rob, could you fix your patch to reflect that? Having a current node shouldn't be mandatory (the information would be incorrect anyway). It's a remnant of the old content management system and it'S now obsolete.

Changed 12 years ago by rob_janes

  • status changed from new to assigned

ok. going back to $node = null.

Changed 11 years ago by max-horvath

  • status changed from assigned to closed
  • resolution set to fixed

Fixed in [1090]

Changed 11 years ago by benoitg

  • component changed from Auth server: Authentication to Auth server, Other
Note: See TracTickets for help on using tickets.