From 344cf5aa862c55a94530628b913d11e190b29508 Mon Sep 17 00:00:00 2001
From: lijing <li_jing@pku.edu.cn>
Date: Mon, 31 Jul 2017 00:12:49 +0800
Subject: [PATCH] add some annotation to the code.

---
 Main/ghttp.cpp         | 10 ++++++++--
 Server/web/index.js    |  2 ++
 Server/web/openkg.html |  3 ++-
 3 files changed, 12 insertions(+), 3 deletions(-)

diff --git a/Main/ghttp.cpp b/Main/ghttp.cpp
index 80c8969..4145c73 100644
--- a/Main/ghttp.cpp
+++ b/Main/ghttp.cpp
@@ -514,6 +514,7 @@ void delete_result(const HttpServer& server, const shared_ptr<HttpServer::Respon
 			else
 			{
 				cout << "file delete." << endl;
+				//Notice: if file deleted successfully, service need to response to the browser, if not the browser will not execute the call-back function.
 				string success = "file delete successfully.";
 				*response << "HTTP/1.1 200 OK\r\nContent-Length: " << success.length() << "\r\n\r\n" << success;
 			}
@@ -555,7 +556,7 @@ void download_result(const HttpServer& server, const shared_ptr<HttpServer::Resp
 			//cout<<"open file!!!"<<endl;
 			auto length=ifs->tellg();
 			ifs->seekg(0, ios::beg);
-
+			//!Notice: remember to set the Content-Disposition and the Content-type to let the browser to download.
 			*response << "HTTP/1.1 200 OK\r\n" << "Content-Length: " << length << "\r\n";
 			*response << "Content-Disposition: attachment; filename=sparql.txt" << "\r\n";
 			*response << "Content-Type: application/octet-stream" << "\r\n\r\n";
@@ -841,7 +842,10 @@ bool query_handler(const HttpServer& server, const shared_ptr<HttpServer::Respon
 			outfile.close();
 			if(rs.ansNum <= 100)
 			{
-				
+				//!Notice: remember to set no-cache in the response of query, Firefox and chrome works well even if you don't set, but IE will act strange if you don't set
+				//beacause IE will defaultly cache the query result after first query request, so the following query request of the same url will not be send if the result in cache isn't expired.
+				//then the following query will show the same result without sending a request to let the service run query
+				//so the download function will go wrong because there is no file in the service.
 				*response << "HTTP/1.1 200 OK\r\nContent-Length: " << ansNum_s.length()+filename.length()+success.length()+4;
 				*response << "\r\nContent-Type: text/plain";
 				*response << "\r\nCache-Control: no-cache" << "\r\nPragma: no-cache" << "\r\nExpires: 0";
@@ -1003,6 +1007,8 @@ bool default_handler(const HttpServer& server, const shared_ptr<HttpServer::Resp
 			ifs->seekg(0, ios::beg);
 
 			*response << "HTTP/1.1 200 OK\r\n" << cache_control << etag << "Content-Length: " << length << "\r\n";
+			//!Notice: IE need to declare the Content-type of the response, if not the browser will not show the result with an unknow format.
+			//it works well in Firefox and Chrome if you don't declare the Content-type, but in IE the css file will be ignored because of unmatched MIME type, then the web page will show in a form without css style and it looks ugly.
 			if(extName == ".html")
 				*response << "Content-Type: text/html" << "\r\n\r\n";
 			else if(extName == ".js")
diff --git a/Server/web/index.js b/Server/web/index.js
index 82925be..e386161 100644
--- a/Server/web/index.js
+++ b/Server/web/index.js
@@ -112,6 +112,8 @@ function query(dp) {
 				var tmp2 = "?operation=delete&filepath=" + fileName;
 				var request2 = escape(tmp2);	
 				var element2 = document.getElementById("back");
+				//!Notice: element2 is a "<a>" tag, and it has two actions, href and onclick, be careful with the executing order of these two actions.
+				//in this case, we use a call-back function to prevent strange things. we return to the origin web page after the request to delete file returns successfully.
 				element2.onclick = function(){
 					$.get(request2, function(data, status){
 						//alert("delete return");
diff --git a/Server/web/openkg.html b/Server/web/openkg.html
index 0bae4a7..ce972eb 100644
--- a/Server/web/openkg.html
+++ b/Server/web/openkg.html
@@ -51,10 +51,11 @@
       <li><a href="http://breastcancer4.gstore-pku.com" class="cc-active">breastcancer4.gstore-pku.com</a></li>
       <li><strong>Clinga</strong></li>
       <li><a href="http://clinga.gstore-pku.com" class="cc-active">clinga.gstore-pku.com</a></li>
-      <li><strong>Person</strong></li>
+      <!--<li><strong>Person</strong></li>
       <li><a href="http://person-zhishime.gstore-pku.com" class="cc-active">person-zhishime.gstore-pku.com</a></li>
       <li><a href="http://person-multype.gstore-pku.com" class="cc-active">person-multype.gstore-pku.com</a></li>
       <li><a href="http://person-taxonomy.gstore-pku.com" class="cc-active">person-taxonomy.gstore-pku.com</a></li>
+	  -->
     </ul>
   </div>
 </section>