website updates and leave balance bugfix

hr/doctype/leave_application/leave_application.py

@@ -64,7 +64,7 @@ class DocType:
 	def validate_balance_leaves(self):
 		if self.doc.from_date and self.doc.to_date and not self.is_lwp():
 			bal = get_leave_balance(self.doc.leave_type, self.doc.employee, 
-				self.doc.fiscal_year)["leave_balance"]
+				self.doc.fiscal_year)
 			tot_leaves = self.get_total_leave_days()
 			bal, tot_leaves = bal, tot_leaves
 			webnotes.conn.set(self.doc, 'leave_balance', flt(bal['leave_balance']))

patches/patch_list.py

@@ -725,4 +725,8 @@ patch_list = [
 		'patch_module': 'patches.december_2012',
 		'patch_file': 'reload_debtors_creditors_ledger',
 	},
+	{
+		'patch_module': 'patches.december_2012',
+		'patch_file': 'website_cache_refactor',
+	},
 ]

website/blog.py

@@ -15,12 +15,11 @@ def get_blog_list(args=None):
 	
 	query = """\
 		select
-			cache.name as name, cache.html as content,
+			name, content, owner, creation as creation,
-			blog.owner as owner, blog.creation as published,
+			title, (select count(name) from `tabComment` where
-			blog.title as title, (select count(name) from `tabComment` where
+				comment_doctype='Blog' and comment_docname=name) as comments
-				comment_doctype='Blog' and comment_docname=blog.name) as comments
+		from `tabBlog`
-		from `tabWeb Cache` cache, `tabBlog` blog
+		where ifnull(published,0)=1
-		where cache.doc_type = 'Blog' and blog.page_name = cache.name
 		order by published desc, name asc"""
 	
 	from webnotes.widgets.query_builder import add_limit_to_query
@@ -35,7 +34,7 @@ def get_blog_list(args=None):
 	for res in result:
 		from webnotes.utils import global_date_format, get_fullname
 		res['full_name'] = get_fullname(res['owner'])
-		res['published'] = global_date_format(res['published'])
+		res['published'] = global_date_format(res['creation'])
 		if not res['content']:
 			res['content'] = website.web_cache.get_html(res['name'])
 		res['content'] = split_blog_content(res['content'])

website/doctype/blog/blog.txt

@@ -1,137 +1,102 @@
-# DocType, Blog
 [
-
-	# These values are common in all dictionaries
  {
-		'creation': '2012-07-27 19:32:53',
+  "owner": "Administrator", 
-		'docstatus': 0,
+  "docstatus": 0, 
-		'modified': '2012-08-03 12:18:36',
+  "creation": "2012-11-30 18:14:18", 
-		'modified_by': u'Administrator',
+  "modified_by": "Administrator", 
-		'owner': u'Administrator'
+  "modified": "2012-12-06 16:33:36"
  }, 
-
-	# These values are common for all DocType
  {
-		'_last_update': u'1328599743',
+  "allow_attach": 1, 
-		'allow_attach': 1,
+  "doctype": "DocType", 
-		'colour': u'White:FFF',
+  "module": "Website", 
-		'doctype': 'DocType',
+  "max_attachments": 5, 
-		'max_attachments': 5,
+  "name": "__common__"
-		'module': u'Website',
-		'name': '__common__',
-		'section_style': u'Simple',
-		'show_in_menu': 0,
-		'version': 1
  }, 
-
-	# These values are common for all DocField
  {
-		'doctype': u'DocField',
+  "name": "__common__", 
-		'name': '__common__',
+  "parent": "Blog", 
-		'parent': u'Blog',
+  "doctype": "DocField", 
-		'parentfield': u'fields',
+  "parenttype": "DocType", 
-		'parenttype': u'DocType'
+  "parentfield": "fields"
  }, 
-
-	# These values are common for all DocPerm
  {
-		'doctype': u'DocPerm',
+  "name": "__common__", 
-		'name': '__common__',
+  "parent": "Blog", 
-		'parent': u'Blog',
+  "read": 1, 
-		'parentfield': u'permissions',
+  "doctype": "DocPerm", 
-		'parenttype': u'DocType',
+  "parenttype": "DocType", 
-		'permlevel': 0,
+  "permlevel": 0, 
-		'read': 1
+  "parentfield": "permissions"
  }, 
-
-	# DocType, Blog
  {
-		'doctype': 'DocType',
+  "name": "Blog", 
-		'name': u'Blog'
+  "doctype": "DocType"
  }, 
-
-	# DocPerm
  {
-		'create': 1,
+  "doctype": "DocField", 
-		'doctype': u'DocPerm',
+  "label": "Title", 
-		'role': u'Website Manager',
+  "fieldname": "title", 
-		'write': 1
+  "fieldtype": "Data", 
+  "reqd": 1, 
+  "permlevel": 0
  }, 
-
-	# DocPerm
  {
-		'create': 1,
+  "doctype": "DocField", 
-		'doctype': u'DocPerm',
+  "label": "Published", 
-		'role': u'Blogger',
+  "fieldname": "published", 
-		'write': 1
+  "fieldtype": "Check", 
+  "permlevel": 0
  }, 
-
-	# DocPerm
  {
-		'doctype': u'DocPerm',
+  "doctype": "DocField", 
-		'role': u'Guest',
+  "label": "Content", 
-		'write': 0
+  "fieldname": "content", 
+  "fieldtype": "Text Editor", 
+  "reqd": 0, 
+  "permlevel": 0
  }, 
-
-	# DocField
  {
-		'doctype': u'DocField',
+  "doctype": "DocField", 
-		'fieldname': u'title',
+  "label": "Page Name", 
-		'fieldtype': u'Data',
+  "fieldname": "page_name", 
-		'label': u'Title',
+  "fieldtype": "Data", 
-		'permlevel': 0,
+  "hidden": 1, 
-		'reqd': 1
+  "permlevel": 1
  }, 
-
-	# DocField
  {
-		'doctype': u'DocField',
+  "doctype": "DocField", 
-		'fieldname': u'published',
+  "label": "Email Sent", 
-		'fieldtype': u'Check',
+  "fieldname": "email_sent", 
-		'label': u'Published',
+  "fieldtype": "Check", 
-		'permlevel': 0
+  "hidden": 1, 
+  "permlevel": 0
  }, 
-
-	# DocField
  {
-		'doctype': u'DocField',
+  "print_hide": 1, 
-		'fieldname': u'content',
+  "no_copy": 1, 
-		'fieldtype': u'Code',
+  "doctype": "DocField", 
-		'label': u'Content',
+  "label": "File List", 
-		'options': u'Markdown',
+  "fieldname": "file_list", 
-		'permlevel': 0,
+  "fieldtype": "Text", 
-		'reqd': 0
+  "hidden": 1, 
+  "permlevel": 0
  }, 
-
-	# DocField
  {
-		'doctype': u'DocField',
+  "create": 1, 
-		'fieldname': u'page_name',
+  "doctype": "DocPerm", 
-		'fieldtype': u'Data',
+  "write": 1, 
-		'hidden': 1,
+  "role": "Website Manager"
-		'label': u'Page Name',
-		'permlevel': 1
  }, 
-
-	# DocField
  {
-		'doctype': u'DocField',
+  "create": 1, 
-		'fieldname': u'email_sent',
+  "doctype": "DocPerm", 
-		'fieldtype': u'Check',
+  "write": 1, 
-		'hidden': 1,
+  "role": "Blogger"
-		'label': u'Email Sent',
-		'permlevel': 0
  }, 
-
-	# DocField
  {
-		'doctype': u'DocField',
+  "write": 0, 
-		'fieldname': u'file_list',
+  "role": "Guest", 
-		'fieldtype': u'Text',
+  "doctype": "DocPerm"
-		'hidden': 1,
-		'label': u'File List',
-		'no_copy': 1,
-		'permlevel': 0,
-		'print_hide': 1
  }
 ]

website/web_cache.py

@@ -39,7 +39,7 @@ def get_page_html(page_name, comments=''):
 
 	if not html:
 		html = load_into_cache(page_name)
-		comments += "\n\npage load status: fresh"
+		comments += "\n\npage load status: cache"
 	
 	# insert comments
 	import webnotes.utils