Update ggsheet.py

PerfecXX · web-flow · commit 9b3351e14eee · 2023-12-13T10:44:14.000+07:00
Fix issues provided by KorkhovPavel

- Improve the start_index and end_index finders in getCell(), getColumn(), and getRow().
- Fix the index range of the returned data from getCell(), getColumn(), and getRow().
- Sync the gen_scriptFile() to script.gs version 0.0.3.
diff --git a/ggsheet.py b/ggsheet.py
@@ -3,7 +3,7 @@
 import ujson
 import os
 
-__version__ = '0.0.2'
+__version__ = '0.0.3'
 __author__ = 'Teeraphat Kullanankanjana'
 
 class MicroGoogleSheet():
@@ -100,15 +100,10 @@ def getCell(self, row=1, column=1):
 
         if responseCode > 0:
             html_code = response.text
-            start_index = html_code.find('<body>')
-            end_index = html_code.find('</body>')
+            start_index = html_code.find('start')
+            end_index = html_code.find('finish')
             script_content = html_code[start_index:end_index]
-            start_index = script_content.find('script type="text/javascript"')
-            end_index = script_content.find('</script>')
-            script_content = script_content[start_index:end_index]
-            script_content = script_content[script_content.find(
-                'body')+18:script_content.find('/body')-21]
-            return script_content
+            return script_content[29:-24]
         else:
             return "Error Code:{}".format(responseCode)
 
@@ -121,15 +116,10 @@ def getRow(self, row=1):
         responseCode = response.status_code
         if responseCode > 0:
             html_code = response.text
-            start_index = html_code.find('<body>')
-            end_index = html_code.find('</body>')
+            start_index = html_code.find('start')
+            end_index = html_code.find('finish')
             script_content = html_code[start_index:end_index]
-            start_index = script_content.find('script type="text/javascript"')
-            end_index = script_content.find('</script>')
-            script_content = script_content[start_index:end_index]
-            script_content = script_content[script_content.find(
-                'body')+18:script_content.find('/body')-21]
-            return script_content
+            return script_content[29:-24]
         else:
             return "Error Code:{}".format(responseCode)
 
@@ -142,15 +132,10 @@ def getColumn(self, column=1):
         responseCode = response.status_code
         if responseCode > 0:
             html_code = response.text
-            start_index = html_code.find('<body>')
-            end_index = html_code.find('</body>')
+            start_index = html_code.find('start')
+            end_index = html_code.find('finish')
             script_content = html_code[start_index:end_index]
-            start_index = script_content.find('script type="text/javascript"')
-            end_index = script_content.find('</script>')
-            script_content = script_content[start_index:end_index]
-            script_content = script_content[script_content.find(
-                'body')+18:script_content.find('/body')-21]
-            return script_content
+            return script_content[29:-24]
         else:
             return "Error Code:{}".format(responseCode)
     
@@ -183,22 +168,31 @@ def deleteColumn(self,column=1):
 
     def gen_scriptFile(self):
         code = """
+/* 
+Author: Teeraphat Kullanankanjana
+Version: 0.0.3
+*/
+
 function doGet(e) {
+  // Extract parameters from the request
   var sheet_id = e.parameter.sheet_id;
   var sheet_name = e.parameter.sheet_name;
   var mode = e.parameter.mode;
-
+  
+  // Open the spreadsheet and get the sheet
   var ss = SpreadsheetApp.openById(sheet_id);
   var sheet = ss.getSheetByName(sheet_name);
-
+  
+  // Update a single cell
   if (mode == "updateCell") {
     var row = e.parameter.row;
     var column = e.parameter.column;
     var data = e.parameter.data;
-
     var cell = sheet.getRange(row, column);
     cell.setValue(data);
   }
+  
+  // Update a row with multiple values
   else if (mode == "updateRow") {
     var row = e.parameter.row;
     var data = [];
@@ -214,7 +208,8 @@ def gen_scriptFile(self):
     var range = sheet.getRange(row, 1, 1, data.length);
     range.setValues([data]);
   }
-
+  
+  // Append a row with multiple values
   else if (mode == "appendRow") {
     var data = [];
     var count = 0;
@@ -235,7 +230,8 @@ def gen_scriptFile(self):
     }
     sheet.getRange(lastRow + 1, 1, 1, data.length).setValues([data]);
   }
-
+  
+  // Append a column with multiple values
   else if (mode == "appendColumn") {
     var data = [];
     var count = 0;
@@ -256,6 +252,8 @@ def gen_scriptFile(self):
     }
     sheet.getRange(1, lastColumn + 1, data.length, 1).setValues(data);
   }
+  
+  // Update a column with multiple values
   else if (mode == "updateColumn") {
     var column = e.parameter.column;
     var data = [];
@@ -271,17 +269,20 @@ def gen_scriptFile(self):
     var range = sheet.getRange(1, column, data.length, 1);
     range.setValues(data);
   }
-
+  
+  // Get the value of a specific cell
   else if (mode == "getCell") {
     var row = e.parameter.row;
     var column = e.parameter.column;
 
     var cell = sheet.getRange(row, column);
     var value = cell.getValue();
 
-    var html = "<html><head><title>Get The data </title></head><body><h1>" + value + "</h1></body></html>";
+    var html = "<html><head><title>Get The data </title></head><body><h1>start</h1><h1>" + value + "</h1><h1>finish</h1></body></html>";
     return HtmlService.createHtmlOutput(html);
   }
+  
+  // Get the values of a specific row
   else if (mode == "getRow") {
   var row = e.parameter.row;
   var range = sheet.getRange(row, 1, 1, sheet.getLastColumn());
@@ -292,9 +293,11 @@ def gen_scriptFile(self):
     heading += values[i] + " ";
   }
 
-  var html = "<html><head><title>Get Row Data</title></head><body><h1>" + heading + "</h1></body></html>";
+  var html = "<html><head><title>Get Row Data</title></head><body><h1>start</h1><h1>" + heading + "</h1><h1>finish</h1></body></html>";
   return HtmlService.createHtmlOutput(html);
 }
+ 
+ // Get the values of a specific column
 else if (mode == "getColumn") {
   var column = e.parameter.column;
   var range = sheet.getRange(1, column, sheet.getLastRow(), 1);
@@ -304,20 +307,24 @@ def gen_scriptFile(self):
     heading += values[i] + " ";
   }
 
-  var html = "<html><head><title>Get Row Data</title></head><body><h1>" + heading + "</h1></body></html>";
+  var html = "<html><head><title>Get Column Data</title></head><body><h1>start</h1><h1>" + heading + "</h1><h1>finish</h1></body></html>";
   return HtmlService.createHtmlOutput(html);
   
 }
+  
+  // Delete a specific row
 else if (mode == "deleteRow") {
   var row = e.parameter.row;
   sheet.deleteRow(row);
 }
 
+// Delete a specific column
 else if (mode == "deleteColumn") {
   var column = e.parameter.column;
   sheet.deleteColumn(column);
 }
 
+// Clear the content of a specific cell
 else if (mode == "deleteCell") {
   var row = e.parameter.row;
   var column = e.parameter.column;
@@ -328,3 +335,4 @@ def gen_scriptFile(self):
         """
         with open('script.txt', 'w') as file:
             file.write(code)
+
