All menus now should be instances, not classes anymore, this will avo…

…id mistakes

README.md

@@ -64,9 +64,9 @@ class BankMenu(alacarte.Menu):
 	group = 'main'
 	label = 'Bank'
 	submenus = (
-		BankTransactionsMenu,
-		BankBalanceMenu,
-		BankPremiumMenu,
+		BankTransactionsMenu(),
+		BankBalanceMenu(),
+		BankPremiumMenu(),
 	)
 
 	def shown(self):

alacarte/__init__.py

@@ -14,12 +14,15 @@
 
 
 def register(menu_class):
+	# Instantiate the menu class before registering it
+	menu = menu_class()
+
 	try:
-		group = menu_class.group
+		group = menu.group
 	except AttributeError:
 		group = ''
 
-	MENU_REGISTRY[group].append(menu_class)
+	MENU_REGISTRY[group].append(menu)
 
 
 def get_menus(group):

alacarte/menus.py

@@ -2,36 +2,33 @@
 
 
 class Menu:
-	@classmethod
-	def shown(cls):
+	def shown(self):
 		"""
 		All menus are shown by default.
 		Override this method to implement custom behavior.
 		"""
 		return True
 
-	@classmethod
-	def url(cls):
+	def url(self):
 		"""
 		Try to reverse `url_name`, fallback to '#' if not possible.
 		"""
 		try:
-			return reverse(cls.url_name)
+			return reverse(self.url_name)
 		except AttributeError:
 			return '#'
 		except NoReverseMatch:
 			raise
 
-	@classmethod
-	def is_active(cls):
+	def is_active(self):
 		"""
 		A menu is active either if its `url_name` is the current or if
 		any of its `submenus` are active.
 		"""
 		url = sub_urls = False
-		if hasattr(cls, 'url_name'):
-			url = reverse(cls.url_name) == cls.context['request'].path
-		if hasattr(cls, 'submenus'):
-			sub_urls = any([s.is_active() for s in cls.submenus])
+		if hasattr(self, 'url_name'):
+			url = reverse(self.url_name) == self.context['request'].path
+		if hasattr(self, 'submenus'):
+			sub_urls = any([s.is_active() for s in self.submenus])
 
 		return url or sub_urls