Use better variable names.

README.md

@@ -11,14 +11,15 @@ Hui Lan <hui.lan@cantab.net>
 
 
 EnglishPal allows the user to build his list of new English words
-picked from articles selected for him to read according his vocabulary level.  EnglishPal will determine a user's vocabulary level based on his picked words.  After that, it will recommend articles for him to read, in order to booster his English vocabulary furthermore.
+picked from articles selected for him according his vocabulary level.
 
 
-## Run on your own laptop
+## Run it on a local machine
+
 
 `python3 main.py`
 
-Make sure you have put the SQLite database file in the path `app/static` (see below).
+Make sure you have the SQLite database file in `app/static` (see below).
 
 
 ## Run it as a Docker container
@@ -28,32 +29,32 @@ Assuming that docker has been installed and that you are a sudo user (i.e., sudo
 
 `sudo ./build.sh`
 
-Open your favourite Internet browser and enter this URL address: `http://ip-address:90`.  Note: you must update the variable `DEPLOYMENT_DIR` in `build.sh`.
+Open your favourite Internet browser and enter this URL address: `http://ip-address:90`.
 
 ### Explanation on the commands in build.sh
 
-My steps for deploying English on a Ubuntu server.
+My steps for deploying English on the server.
 
 - ssh to ubuntu@118.*.*.118
 
-- cd to `/home/lanhui/englishpal2/EnglishPal`
+- cd to /home/lanhui/englishpal2/EnglishPal
 
 - Stop all docker service: `sudo service docker restart`.  If you know the docker container ID, then the above command is an overkill.  Use the following command instead: `sudo docker stop ContainerID`.  You could get all container IDs with the following command: `sudo docker ps`
 
-- Rebuild container. Run the following command to rebuild a docker image each time after the source code gets updated: `sudo docker build -t englishpal .`
+- Rebuild container. Run the following command to rebuild a docker image after the code gets updated: `sudo docker build -t englishpal .`
 
-- Run the application: `sudo docker run -d -p 90:80 -v /home/lanhui/englishpal2/EnglishPal/app/static/frequency:/app/static/frequency -t englishpal`. If you use `sudo docker run -d -p 90:80 -t englishpal`, data will be lost after terminating the program.  If you want to automatically restart the docker image after each system reboot, add the option `--restart=always` after `docker run`.
+- Run the application: `sudo docker run -d -p 90:80 -v /home/lanhui/englishpal2/EnglishPal/app/static/frequency:/app/static/frequency -t englishpal`. If you use `sudo docker run -d -p 90:80 -t englishpal`, data will be lost after terminating the program.
 
-- Save disk space: `sudo docker system prune -a -f`
-
-`build.sh` contains all the above commands.  Run "sudo ./build.sh" to rebuild and start the web application.
+- Save space: `sudo docker system prune -a -f`
 
 
-#### Other useful docker commands
+### Other useful docker commands
 
 - `sudo docker ps -a`
 
-- `sudo docker logs image_name`, where `image_name` could be obtained from `sudo docker ps`.
+- `sudo docker logs image_name`, where image_name could be obtained from `sudo docker ps`.
+
+`build.sh` contains all the above commands.  Run "sudo ./build.sh" to rebuild and run the web application.
 
 
 
@@ -67,10 +68,6 @@ All articles are stored in the `article` table in a SQLite file called
 
 To add articles, open and edit `app/static/wordfreqapp.db` using DB Browser for SQLite (https://sqlitebrowser.org).
 
-### Extending an account's expiry date
-
-By default, an account's expiry is 30 days after first sign-up.  To extend account's expiry date, open and edit `user` table in `app/static/wordfreqapp.db`.  Simply update field `expiry_date`.
-
 ### Exporting the database
 
 Export wordfreqapp.db to wordfreqapp.sql using the following commands:
@@ -95,31 +92,33 @@ sqlite3 wordfreqapp.db`.  Delete wordfreqapp.db first if it exists.
 ### Uploading wordfreqapp.db to the server
 
 
-`pscp wordfreqapp.db lanhui@118.*.*.118:/home/lanhui/englishpal2/EnglishPal/app/static`
+`pscp wordfreqapp.db lanhui@118.*.*.118:/home/lanhui/englishpal/app/static`
 
 
 
 ## Feedback
 
-We welcome feedback on EnglishPal.  Feedback examples:
+We welcome feedback on EnglishPal.
 
-### Feedback 1
-
-- "Need a phone app.  I use phone a lot.  You cannot ask students to use computers."
+### Respondent 1
 
 
-### Feedback 2
+"Need a phone app.  I use phone a lot.  You cannot ask students to use computers."
+
+Can take a picture for text.  Automatic translation.
+
+### Respondent 2
 
 
-- “成为会员”改成“注册”
+“成为会员”改成“注册”
 
-- “登出”改成“退出”
+“登出”改成“退出”
 
-- “收集生词吧”改成“生词收集栏”
+“收集生词吧”改成“生词收集栏”
 
-- 不要自动显示下一篇
+“不要自动显示下一篇”
 
-- 需要有“上一篇”、“下一篇”按钮。
+需要有“上一篇”、“下一篇”
 
 
 
@@ -138,7 +137,7 @@ EnglishPal's bugs and improvement suggestions are recorded in [Bugzilla](http://
 - Usability testing
 
 
-## Improvements made by contributors (incomplete list)
+## Improvements made by contributors
 
 
 ### 朱文绮
@@ -160,6 +159,7 @@ too many words that they already know, on the other hand, it can
 reduce unnecessary memory occupied by the database, in addition, it
 can also improve the simplicity of the page.
 
+More information at: http://118.25.96.118/kanboard/?controller=TaskViewController&action=readonly&task_id=736&token=81a561da57ff7a172da17a480f0d421ff3bc69efbd29437daef90b1b8959
 
 
 ### 占健豪
@@ -181,14 +181,4 @@ Demo video link: https://b23.tv/QuB77m
 Bug report: http://118.25.96.118/bugzilla/show_bug.cgi?id=215
 
 
-
-### 丁锐
-
-修复了以下漏洞
-
-漏洞：新用户在创建账号时，不需要输入确定密码也可以注册成功，并且新账户可以正常使用。
-
-Bug report: http://118.25.96.118/bugzilla/show_bug.cgi?id=489
-
-
-*Last modified on 2023-01-30*
+*Last modified on 2021-10-17*

app/Article.py

@@ -68,7 +68,7 @@ def get_today_article(user_word_list, articleID):
     s += '<div class="p-3 mb-2 bg-light text-dark">'
     article_title = get_article_title(d['text'])
     article_body = get_article_body(d['text'])
-    s += '<p class="display-5">%s</p>' % (article_title)
+    s += '<p class="display-3">%s</p>' % (article_title)
     s += '<p class="lead"><font id="article" size=2>%s</font></p>' % (article_body)
     s += '<p><small class="text-muted">%s</small></p>' % (d['source'])
     s += '<p><b>%s</b></p>' % (get_question_part(d['question']))

app/Login.py

@@ -1,6 +1,5 @@
 import hashlib
-import string
-from datetime import datetime, timedelta
+from datetime import datetime
 from UseSqlite import InsertQuery, RecordQuery
 
 path_prefix = '/var/www/wordfreq/wordfreq/'
@@ -23,7 +22,7 @@ def verify_user(username, password):
 
 def add_user(username, password):
     start_date = datetime.now().strftime('%Y%m%d')
-    expiry_date = (datetime.now() + timedelta(days=30)).strftime('%Y%m%d') # will expire after 30 days
+    expiry_date = '20221230'
     # 将用户名和密码一起加密，以免暴露不同用户的相同密码
     password = md5(username + password)
     rq = InsertQuery(path_prefix + 'static/wordfreqapp.db')
@@ -82,32 +81,3 @@ def md5(s):
     '''
     h = hashlib.md5(s.encode(encoding='utf-8'))
     return h.hexdigest()
-
-
-class UserName:
-    def __init__(self, username):
-        self.username = username
-
-    def validate(self):
-        if len(self.username) > 20:
-            return f'{self.username} is too long.  The user name cannot exceed 20 characters.'
-        if self.username.startswith('.'): # a user name must not start with a dot
-            return 'Period (.) is not allowed as the first letter in the user name.'
-        if ' ' in self.username: # a user name must not include a whitespace
-            return 'Whitespace is not allowed in the user name.'
-        for c in self.username: # a user name must not include special characters, except non-leading periods or underscores
-            if c in string.punctuation and c is not '.' and c is not '_':
-                return f'{c} is not allowed in the user name.'
-        if self.username in ['signup', 'login', 'logout', 'reset', 'mark', 'back', 'unfamiliar', 'familiar', 'del']:
-            return 'You used a restricted word as your user name.  Please come up with a better one.'
-
-        return 'OK'
-
-
-class WarningMessage:
-    def __init__(self, s):
-        self.s = s
-
-    def __str__(self):
-        return UserName(self.s).validate()
-

app/account_service.py

@@ -1,10 +1,10 @@
 from flask import *
-from Login import check_username_availability, verify_user, add_user, get_expiry_date, change_password, WarningMessage
-
+from Login import check_username_availability, verify_user, add_user, get_expiry_date, change_password
 
 # 初始化蓝图
 accountService = Blueprint("accountService", __name__)
 
+
 ### Sign-up, login, logout ###
 @accountService.route("/signup", methods=['GET', 'POST'])
 def signup():
@@ -19,21 +19,13 @@ def signup():
         # POST方法需判断是否注册成功，再根据结果返回不同的内容
         username = escape(request.form['username'])
         password = escape(request.form['password'])
-        password2 = escape(request.form['password2'])
-        
-        #! 添加如下代码为了过滤注册时的非法字符
-        warn = WarningMessage(username)
-        if str(warn) != 'OK':
-            return str(warn)
-        
+
         available = check_username_availability(username)
         if not available: # 用户名不可用
             flash('用户名 %s 已经被注册。' % (username))
             return render_template('signup.html')
         elif len(password.strip()) < 4: # 密码过短
             return '密码过于简单。'
-        elif password != password2:
-            return '确认密码与输入密码不一致！'
         else: # 添加账户信息
             add_user(username, password)
             verified = verify_user(username, password)
@@ -50,7 +42,6 @@ def signup():
                 return '用户名密码验证失败。'
 
 
-
 @accountService.route("/login", methods=['GET', 'POST'])
 def login():
     '''
@@ -115,13 +106,6 @@ def reset():
         # POST请求用于提交修改后信息
         old_password = escape(request.form['old-password'])
         new_password = escape(request.form['new-password'])
-
-        re_new_password = escape(request.form['re-new-password'])  # 确认新密码
-        if re_new_password != new_password: #验证新密码两次输入是否相同
-            return '新密码不匹配，请重新输入'
-        if len(new_password) < 4: #验证新密码长度，原则参照注册模块
-            return '密码过于简单。(密码长度至少4位)'
-
         flag = change_password(username, old_password, new_password) # flag表示是否修改成功
         if flag:
             session['logged_in'] = False
@@ -142,4 +126,4 @@ alert('密码修改失败');
 window.location.href="/reset";
 </script>
 
-'''
+'''

app/main.py

@@ -6,7 +6,6 @@
 # Written permission must be obtained from the author for commercial uses.
 ###########################################################################
 
-from flask import escape
 from Login import *
 from Article import *
 import Yaml
@@ -82,7 +81,7 @@ def mainpage():
     :return: 主界面
     '''
     if request.method == 'POST':  # when we submit a form
-        content = escape(request.form['content'])
+        content = request.form['content']
         f = WordFreq(content)
         lst = f.get_freq()
         # save history

app/static/css/bootstrap.css

@@ -417,7 +417,7 @@ progress {
 }
 
 .lead {
-    font-size: 2rem;
+    font-size: 1.25rem;
     font-weight: 300
 }
 

app/static/css/login_service.css

@@ -1,107 +0,0 @@
-/*样式应用于login、signup、reset三个页面*/
-
-.container {
-    background-color: #FFFFFF;
-    width: 400px;
-    height: 500px;
-    margin: 7em auto;
-    border-radius: 1.5em;
-    box-shadow: 0px 11px 35px 2px rgba(0, 0, 0, 0.14);
-}
-
-/*增加一个类reset-heading*/
-.signin-heading, .reset-heading {
-    padding-top: 5px;
-    color: #8C55AA;
-    font-family: 'Ubuntu', sans-serif;
-    font-weight: bold;
-    font-size: 23px;
-    text-align: center;
-}
-
-/*增加2个类.old-password和.new-password*/
-.username, .email, .password, .re-password, .old-password, .new-password,.re-new-password {
-    width: 76%;
-    color: rgb(38, 50, 56);
-    font-weight: 700;
-    font-size: 14px;
-    letter-spacing: 1px;
-    background: rgba(136, 126, 126, 0.04);
-    padding: 10px 20px;
-    border: none;
-    border-radius: 20px;
-    outline: none;
-    box-sizing: border-box;
-    border: 2px solid rgba(124, 16, 97, 0.02);
-    margin-bottom: 50px;
-    margin-left: 46px;
-    text-align: center;
-    margin-bottom: 27px;
-    font-family: 'Ubuntu', sans-serif;
-}
-
-.btn {
-    width: 50%;
-    border: none;
-    border-radius: 20px;
-    box-sizing: border-box;
-    border: 2px solid #8C55AA;
-    margin-bottom: 50px;
-    margin-left: 90px;
-    padding: 10px 20px;
-
-}
-
-.btn:hover {
-
-    background: #8C55AA;
-    transition: .5s;
-    cursor: pointer;
-    color: #fff;
-}
-
-
-.signup {
-    display: flex;
-    justify-content: center;
-    align-items: center;
-}
-
-ul {
-    position: absolute;
-    display: flex;
-    left: 65%;
-
-}
-
-li {
-    padding: 10px;
-    margin: 10px;
-}
-
-a {
-    text-decoration: none;
-    list-style: none;
-    font-weight: bold;
-    font-family: 'ink free';
-
-}
-
-.main_menu a {
-    color: #fff;
-    font-size: 300px;
-}
-
-li :hover {
-    color: #8C55AA;
-    transition: .5s;
-}
-
-h1 {
-    font-family: 'ink free';
-
-}
-
-.main_menu h1 {
-    color: #fff;
-}

app/static/frequency/README.txt

@@ -1,5 +0,0 @@
-This folder holds users' vocabulary files.
-Each file ends with .pickle.
-For example, mrlan.pickle is the vocabulary file for user mrlan.
-
-

app/static/js/fillword.js

@@ -62,7 +62,3 @@ function onReadClick() {
 function onChooseClick() {
     isChoose = !isChoose;
 }
-
-function stopRead() {
-    reader.cancel();
-}

app/static/js/highlight.js

@@ -1,7 +1,7 @@
 let isHighlight = true;
 
 function cancelBtnHandler() {
-    cancelHighlighting();
+    cancel_highLight();
     document.getElementById("text-content").removeEventListener("click", fillInWord, false);
     document.getElementById("text-content").removeEventListener("touchstart", fillInWord, false);
     document.getElementById("text-content").addEventListener("click", fillInWord2, false);
@@ -22,53 +22,54 @@ function getWord() {
 
 function highLight() {
     if (!isHighlight) return;
-    let articleContent = document.getElementById("article").innerText; //将原来的.innerText改为.innerHtml，使用innerText会把原文章中所包含的<br>标签去除，导致处理后的文章内容失去了原来的格式
-    let pickedWords = document.getElementById("selected-words");  // words picked to the text area
-    let dictionaryWords = document.getElementById("selected-words2"); // words appearing in the user's new words list
-    let allWords = "";  //初始化allWords的值，避免进入判断后编译器认为allWords未初始化的问题
-    if(dictionaryWords != null){//增加一个判断，检查生词本里面是否为空，如果为空，allWords只添加选中的单词
-        allWords = pickedWords.value + " " + dictionaryWords.value;
-    }
-    else{
-        allWords = pickedWords.value + " ";
-    }
-    const list = allWords.split(" ");//将所有的生词放入一个list中，用于后续处理
-    for (let i = 0; i < list.length; ++i) {
-        list[i] = list[i].replace(/(^\s*)|(\s*$)/g, ""); //消除单词两边的空字符
-        list[i] = list[i].replace('|', "");
-        list[i] = list[i].replace('?', "");
-        if (list[i] !== "" && "<mark>".indexOf(list[i]) === -1 && "</mark>".indexOf(list[i]) === -1) {
-	    //将文章中所有出现该单词word的地方改为：" <mark>" + word + "<mark> "。 正则表达式RegExp()中，"\\s"代表单词前后必须要有空格，以防止只对单词中的部分字符高亮的情况出现。
-            articleContent = articleContent.replace(new RegExp("\\s"+list[i]+"\\s", "g"), " <mark>" + list[i] + "</mark> ");
-        }
-    }
-    document.getElementById("article").innerHTML = articleContent;
-}
-
-function cancelHighlighting() {
-    let articleContent = document.getElementById("article").innerText;//将原来的.innerText改为.innerHtml，原因同上
-    let pickedWords = document.getElementById("selected-words");
-    const dictionaryWords = document.getElementById("selected-words2");    
-    const list = pickedWords.value.split(" ");    
-    if (pickedWords != null) {
+    let txt = document.getElementById("article").innerText;
+    let sel_word1 = document.getElementById("selected-words");
+    let sel_word2 = document.getElementById("selected-words2");
+    if (sel_word1 != null) {
+        const list = sel_word1.value.split(" ");
         for (let i = 0; i < list.length; ++i) {
             list[i] = list[i].replace(/(^\s*)|(\s*$)/g, "");
-            if (list[i] !== "") { //原来判断的代码中，替换的内容为“list[i]”这个字符串，这明显是错误的，我们需要替换的是list[i]里的内容
-                articleContent = articleContent.replace(new RegExp("<mark>"+list[i]+"</mark>", "g"), list[i]);
+            if (list[i] !== "" && "<mark>".indexOf(list[i]) === -1 && "</mark>".indexOf(list[i]) === -1) {
+                txt = txt.replace(new RegExp(list[i], "g"), "<mark>" + list[i] + "</mark>");
             }
         }
     }
-    if (dictionaryWords != null) {
-        let list2 = pickedWords.value.split(" ");
+    if (sel_word2 != null) {
+        const list2 = sel_word2.value.split(" ");
         for (let i = 0; i < list2.length; ++i) {
-            list2 = dictionaryWords.value.split(" ");
             list2[i] = list2[i].replace(/(^\s*)|(\s*$)/g, "");
-            if (list2[i] !== "") { //原来代码中，替换的内容为“list[i]”这个字符串，这明显是错误的，我们需要替换的是list[i]里的内容
-                articleContent = articleContent.replace(new RegExp("<mark>"+list2[i]+"</mark>", "g"), list2[i]);
+            if (list2[i] !== "" && "<mark>".indexOf(list2[i]) === -1 && "</mark>".indexOf(list2[i]) === -1) {
+                txt = txt.replace(new RegExp(list2[i], "g"), "<mark>" + list2[i] + "</mark>");
             }
         }
     }
-    document.getElementById("article").innerHTML = articleContent;
+    document.getElementById("article").innerHTML = txt;
+}
+
+function cancel_highLight() {
+    const list = sel_word1.value.split(" ");
+    let txt = document.getElementById("article").innerText;
+    let sel_word1 = document.getElementById("selected-words");
+    const sel_word2 = document.getElementById("selected-words2");
+    if (sel_word1 != null) {
+        for (let i = 0; i < list.length; ++i) {
+            list[i] = list[i].replace(/(^\s*)|(\s*$)/g, "");
+            if (list[i] !== "") {
+                txt = txt.replace("<mark>" + list[i] + "</mark>", "list[i]");
+            }
+        }
+    }
+    if (sel_word2 != null) {
+        let list2 = sel_word1.value.split(" ");
+        for (let i = 0; i < list2.length; ++i) {
+            list2 = sel_word2.value.split(" ");
+            list2[i] = list2[i].replace(/(^\s*)|(\s*$)/g, "");
+            if (list2[i] !== "") {
+                txt = txt.replace("<mark>" + list[i] + "</mark>", "list[i]");
+            }
+        }
+    }
+    document.getElementById("article").innerHTML = txt;
 }
 
 function fillInWord() {
@@ -76,17 +77,18 @@ function fillInWord() {
 }
 
 function fillInWord2() {
-    cancelHighlighting();
+    cancel_highLight();
 }
 
-function toggleHighlighting() {
+function ChangeHighlight() {
     if (isHighlight) {
         isHighlight = false;
-        cancelHighlighting();
+        cancel_highLight();
     } else {
         isHighlight = true;
         highLight();
+
     }
 }
 
-showBtnHandler();
+showBtnHandler();

app/static/js/word_operation.js

@@ -1,173 +1,43 @@
-function familiar(theWord) {
+function familiar(index) {
     let username = $("#username").text();
-    let word = $("#word_" + theWord).text();
-    let freq = $("#freq_" + theWord).text();
+    let word = $("#word_" + index).text();
+    let freq = $("#freq_" + index).text();
     $.ajax({
         type:"GET",
         url:"/" + username + "/" + word + "/familiar",
         success:function(response){
             let new_freq = freq - 1;
-            const allow_move = document.getElementById("move_dynamiclly").checked;
-            if (allow_move) {
-
-                if (new_freq <= 0) {
-                    removeWord(theWord);
-                } else {
-                    renderWord({ word: theWord, freq: new_freq });
-                }
+            if(new_freq <1) {
+                $("#p_" + index).remove();
             } else {
-                if(new_freq <1) {
-                    $("#p_" + theWord).remove();
-                } else {
-                    $("#freq_" + theWord).text(new_freq);
-                }
+                $("#freq_" + index).text(new_freq);
             }
         }
     });
 }
 
-function unfamiliar(theWord) {
+function unfamiliar(index) {
     let username = $("#username").text();
-    let word = $("#word_" + theWord).text();
-    let freq = $("#freq_" + theWord).text();
+    let word = $("#word_" + index).text();
+    let freq = $("#freq_" + index).text();
     $.ajax({
         type:"GET",
         url:"/" + username + "/" + word + "/unfamiliar",
         success:function(response){
             let new_freq = parseInt(freq) + 1;
-            const allow_move = document.getElementById("move_dynamiclly").checked;
-            if (allow_move) {
-                renderWord({ word: theWord, freq: new_freq });
-            } else {
-                $("#freq_" + theWord).text(new_freq);
-            }
+            $("#freq_" + index).text(new_freq);
         }
     });
 }
 
-function delete_word(theWord) {
+function delete_word(index) {
     let username = $("#username").text();
-    let word = theWord.replace('&amp;', '&');
+    let word = $("#word_" + index).text();
     $.ajax({
         type:"GET",
         url:"/" + username + "/" + word + "/del",
         success:function(response){
-            const allow_move = document.getElementById("move_dynamiclly").checked;
-            if (allow_move) {
-                removeWord(theWord);
-            } else {
-                $("#p_" + theWord).remove();
-            }    
+            $("#p_" + index).remove();
         }
     });
 }
-
-/* 
- * interface Word {
- *   word: string,
- *   freq: number
- * }
-* */
-
-/**
- * 传入一个词频HTML元素，将其解析为Word类型的对象
- */
-function parseWord(element) {
-    const word = element
-        .querySelector("a.btn.btn-light[role=button]")  // 获取当前词频元素的词汇元素
-        .innerText  // 获取词汇值;
-    const freq = Number.parseInt(element.querySelector(`#freq_${word}`).innerText);   // 获取词汇的数量
-    return {
-        word,
-        freq
-    };
-}
-
-/**
- * 使用模板将传入的单词转换为相应的HTML字符串
-*/
-function wordTemplate(word) {
-    // 这个模板应当与 templates/userpage_get.html 中的 <p id='p_${word.word}' class="new-word" > ... </p> 保持一致
-    return `<p id='p_${word.word}' class="new-word" >
-        <a id="word_${word.word}"  class="btn btn-light" href='http://youdao.com/w/eng/${word.word}/#keyfrom=dict2.index'
-           role="button">${word.word}</a>
-        ( <a id="freq_${word.word}" title="${word.word}">${word.freq}</a> )
-        <a class="btn btn-success" onclick="familiar('${word.word}')" role="button">熟悉</a>
-        <a class="btn btn-warning" onclick="unfamiliar('${word.word}')" role="button">不熟悉</a>
-        <a class="btn btn-danger" onclick="delete_word('${word.word}')" role="button">删除</a>
-    </p>`;
-}
-
-/**
- * 删除某一词频元素
- * 此处word为词频元素对应的单词
- */
-function removeWord(word) {
-    // 根据词频信息删除元素
-    word = word.replace('&amp;', '&');
-    const element_to_remove = document.getElementById(`p_${word}`);
-    if (element_to_remove != null) {
-        element_to_remove.remove();
-    }
-}
-
-function renderWord(word) {
-    const container = document.querySelector(".word-container");
-    // 删除原有元素
-    removeWord(word.word);
-    // 插入新元素
-    let inserted = false;
-    const new_element = elementFromString(wordTemplate(word));
-    for (const current of container.children) {
-        const cur_word = parseWord(current);
-        // 找到第一个词频比它小的元素，插入到这个元素前面
-        if (compareWord(cur_word, word) == -1) {
-            container.insertBefore(new_element, current);
-            inserted = true;
-            break;
-        }
-    }
-    // 当word就是词频最小的词时，把他补回去
-    if (!inserted) {
-        container.appendChild(new_element);
-    }
-    // 让发生变化的元素抖动
-    new_element.classList.add("shaking");
-    // 移动到该元素
-    new_element.scrollIntoView({behavior: "smooth", block: "center", inline: "nearest"});
-    // 抖动完毕后删除抖动类
-    setTimeout(() => {
-        new_element.classList.remove("shaking");
-    }, 1600);
-}
-
-/**
- * 从string中创建一个HTML元素并返回
- */
-function elementFromString(string) {
-    const d = document.createElement('div');
-    d.innerHTML = string;
-    return d.children.item(0);
-}
-
-/**
- * 对比两个单词：
- *  当first小于second时返回-1
- *  当first等于second时返回0
- *  当first大于second时返回1
- */
-function compareWord(first, second) {
-    if (first.freq < second.freq) {
-        return -1;
-    }
-    if (first.freq > second.freq) {
-        return 1;
-    }
-    if (first.word < second.word) {
-        return -1;
-    }
-    if (first.word > second.word) {
-        return 1;
-    }
-    return 0;
-}

app/templates/expiry.html

@@ -5,7 +5,7 @@
     <title>账号过期</title>
 </head>
 <body>
-    <p>您的账号过期（过期日 {{expiry_date}}）。</p>
+    <p>您的账号{{ username }}过期。</p>
     <p>为了提高服务质量，English Pal 收取会员费用， 每天1元。</p>
     <p>请决定你要试用的时间长度，扫描下面支付宝二维码支付。 支付时请注明<i>English Pal Membership Fee</i>。 我们会于12小时内激活账号。</p>
     <p><img src="static/donate-the-author-hidden.jpg" width="120px" alt="支付宝二维码" /></p>

app/templates/login.html

@@ -5,24 +5,17 @@ You're logged in already!
 
 {% else %}
 <meta charset="utf-8" name="viewport" content="width=device-width, initial-scale=1.0, minimum-scale=0.5, maximum-scale=3.0, user-scalable=yes" />
-<link rel="stylesheet" href="static/css/login_service.css">
-
-<div class="container">
-
-  <section class="signin-heading">
-    <h1>Sign In</h1>
-  </section>
-
-  <form action="/login" method="POST">
-    <input type="text" placeholder="用户名" class="username" name="username" required>
-    <input type="password" placeholder="密码" class="password"  name="password" required>
-    <button type="submit" class="btn">登录</button>
-  </form>
-
-</div>
-
-<a href="/signup" class="signup">注册</a>
-
+<form action="/login" method="POST">
+  <p>
+      <input type="username" name="username" placeholder="邮箱地址、电话号码">
+  </p>
+  <p>
+      <input type="password" name="password" placeholder="密码">
+  </p>
+  <p>
+      <input type="submit" value="登录">
+  </p>
+</form>
 {% endif %}
 {% endblock %}
 

app/templates/reset.html

@@ -1,22 +1,14 @@
-{% block body %}
-    <meta charset="utf-8" name="viewport"
-          content="width=device-width, initial-scale=1.0, minimum-scale=0.5, maximum-scale=3.0, user-scalable=yes"/>
-    <link rel="stylesheet" href="static/css/login_service.css">
-
-    <div class="container">
-
-        <section class="reset-heading">
-            <h1>Reset Password</h1>
-        </section>
-
-        <form action="/reset" method="POST">
-            <input type="password" placeholder="原密码" class="old-password" name="old-password" required>
-            <input type="password" placeholder="新密码" class="new-password" name="new-password" required>
-            <input type="password" placeholder="确认新密码" class="re-new-password" name="re-new-password" required>
-            <input type="submit" name="submit" class="btn" value="提交"/>
-            <input type="button" name="submit" class="btn" value="放弃修改"
-                   onclick="window.location.href='/{{ username }}'"/>
-        </form>
-
-    </div>
-{% endblock %}
+<html>
+    <body>
+    <form action="/reset" method='POST'>
+        旧密码:
+        <input type="password" name="old-password" />
+        <br/>
+        新密码:
+        <input type="password" name="new-password" />
+        <br/>
+        <input type="submit" name="submit" value="提交" />
+        <input type="button" name="submit" value="放弃修改" onclick="window.location.href='/{{ username }}'"/>
+    </form>
+    </body>
+</html>

app/templates/signup.html

@@ -5,25 +5,15 @@ You're logged in already! <a href="/logout">Logout</a>.
 
 {% else %}
 <meta name="viewport" content="width=device-width, initial-scale=1.0, minimum-scale=0.5, maximum-scale=3.0, user-scalable=yes" />
-<link rel="stylesheet" href="static/css/login_service.css">
 <p>{{ get_flashed_messages()[0] | safe }}</p>
 
+<p>Sign up here.</p>
 
-<div class="container">
-
-  <section class="signin-heading">
-    <h1>Sign Up</h1>
-  </section>
-
-  <form action="/signup" method="POST">
-    <p><input type="username" name="username" placeholder="输入用户名" required="required" class="username"></p>
-    <p><input type="password" name="password" placeholder="输入密码" required="required" class="password"></p>
-    <p><input type="password" name="password2" placeholder="确认密码" required="required" class="password" ></p>
-    <button type="submit" class="btn">注册</button>
-  </form>
-
-</div>
-  
+<form action="/signup" method="POST">
+  <p><input type="username" name="username" placeholder="邮箱地址、电话号码" required="required"></p>
+  <p><input type="password" name="password" placeholder="密码"></p>
+  <p><input type="submit" value="注册"></p>
+</form>
 {% endif %}
 {% endblock %}
 

app/templates/userpage_get.html

@@ -19,20 +19,6 @@
     {% endif %}
 
     <title>EnglishPal Study Room for {{ username }}</title>
-
-    <style>
-        .shaking {
-            animation: shakes 1600ms ease-in-out;
-        }
-
-        @keyframes shakes {
-            10%, 90% { transform: translate3d(-1px, 0, 0); }
-            20%, 50% { transform: translate3d(+2px, 0, 0); }
-            30%, 70% { transform: translate3d(-4px, 0, 0); }
-            40%, 60% { transform: translate3d(+4px, 0, 0); }
-            50% { transform: translate3d(-4px, 0, 0); }
-        }
-    </style>
 </head>
 <body>
 <div class="container-fluid">
@@ -42,17 +28,11 @@
     </p>
     {{ flashed_messages|safe }}
 
-    <a class="btn btn-success" href="/{{ username }}/reset" role="button"> 下一篇 Next Article </a>    
-    {% if session.get('articleID') != session.get('old_articleID') %}
-        {% if session.get('old_articleID') != None %}
-            <a class="btn btn-success" href="/{{ username }}/back" role="button"> 上一篇 Previous Article </a>
-        {% endif%}
-    {% endif %}
-
+    <p><a class="btn btn-success" href="/{{ username }}/reset" role="button"> 下一篇 Next Article </a></p>
     <p><b>阅读文章并回答问题</b></p>
     <div id="text-content">{{ today_article|safe }}</div>
 
-    <input type="checkbox" onclick="toggleHighlighting()" checked/>生词高亮
+    <input type="checkbox" onclick="ChangeHighlight()" checked/>生词高亮
     <input type="checkbox" onclick="onReadClick()" checked/>大声朗读
     <input type="checkbox" onclick="onChooseClick()" checked/>划词入库
     <div class="range">
@@ -81,30 +61,22 @@
     {% endif %}
 
     {% if d_len > 0 %}
-        <p>
-            <b>我的生词簿</b> 
-            <label for="move_dynamiclly">
-                <input type="checkbox" name="move_dynamiclly" id="move_dynamiclly" checked>
-                允许动态调整顺序
-            </label>
-        </p>
-        <a name="aaa"></a>
-        <div class="word-container">
-            {% for x in lst3 %}
-                {% set word = x[0] %}
-                {% set freq = x[1] %}
-                {% if session.get('thisWord') == x[0] and session.get('time') == 1 %}
-                {% endif %}
-                <p id='p_{{ word }}' class="new-word" >
-                    <a id="word_{{ word }}"  class="btn btn-light" href='http://youdao.com/w/eng/{{ word }}/#keyfrom=dict2.index'
-                    role="button">{{ word }}</a>
-                    ( <a id="freq_{{ word }}" title="{{ word }}">{{ freq }}</a> )
-                    <a class="btn btn-success" onclick="familiar('{{ word }}')" role="button">熟悉</a>
-                    <a class="btn btn-warning" onclick="unfamiliar('{{ word }}')" role="button">不熟悉</a>
-                    <a class="btn btn-danger" onclick="delete_word('{{ word }}')" role="button">删除</a>
-                </p>
-            {% endfor %}
-        </div>
+        <p><b>我的生词簿</b></p>
+        {% for x in lst3 %}
+            {% set word = x[0] %}
+            {% set freq = x[1] %}
+            {% if session.get('thisWord') == x[0] and session.get('time') == 1 %}
+                <a name="aaa"></a>
+            {% endif %}
+            <p id='p_{{ loop.index0 }}' class="new-word" >
+                <a id="word_{{ loop.index0 }}"  class="btn btn-light" href='http://youdao.com/w/eng/{{ word }}/#keyfrom=dict2.index'
+                   role="button">{{ word }}</a>
+                ( <a id="freq_{{ loop.index0 }}" title="{{ word }}">{{ freq }}</a> )
+                <a class="btn btn-success" onclick="familiar({{ loop.index0 }})" role="button">熟悉</a>
+                <a class="btn btn-warning" onclick="unfamiliar({{ loop.index0 }})" role="button">不熟悉</a>
+                <a class="btn btn-danger" onclick="delete_word({{ loop.index0 }})" role="button">删除</a>
+            </p>
+        {% endfor %}
         <input id="selected-words2" type="hidden" value="{{ words }}">
     {% endif %}
 </div>

app/templates/userpage_post.html

@@ -20,7 +20,7 @@
     <title>EnglishPal Study Room for {{username}}</title>
 </head>
 <body>
-     <p>取消勾选认识的单词</p>
+     <p>勾选不认识的单词</p>
       <form method="post" action="/{{username}}/mark">
        <input type="submit" name="add-btn" value="加入我的生词簿"/>
        {% for x in lst %}
@@ -30,7 +30,7 @@
             :
             <a href='http://youdao.com/w/eng/{{word}}/#keyfrom=dict2.index' title={{word}}>{{word}}</a>
             ({{x[1]}})
-            <input type="checkbox" name="marked" value="{{word}}" checked>
+            <input type="checkbox" name="marked" value={{word}}>
         </p>
 
        {% endfor %}
@@ -42,4 +42,4 @@
         {% endfor %}
     {% endif %}
 </body>
-</html>
+</html>

app/user_service.py

@@ -29,25 +29,12 @@ def user_reset(username):
     :param username: 用户名
     :return: 返回页面内容
     '''
-    session['old_articleID'] = session.get('articleID')
     if request.method == 'GET':
         session['articleID'] = None
         return redirect(url_for('user_bp.userpage', username=username))
     else:
         return 'Under construction'
 
-@userService.route("/<username>/back", methods=['GET'])
-def user_back(username):
-    '''
-    用户界面
-    :param username: 用户名
-    :return: 返回页面内容
-    '''
-    if request.method == 'GET':
-        session['articleID'] = session.get('old_articleID')
-        return redirect(url_for('user_bp.userpage', username=username))
-
-
 
 @userService.route("/<username>/<word>/unfamiliar", methods=['GET', 'POST'])
 def unfamiliar(username, word):
@@ -107,7 +94,7 @@ def userpage(username):
     # 用户过期
     user_expiry_date = session.get('expiry_date')
     if datetime.now().strftime('%Y%m%d') > user_expiry_date:
-        return render_template('expiry.html', expiry_date=user_expiry_date)
+        return render_template('expiry.html')
 
     # 获取session里的用户名
     username = session.get('username')
@@ -115,7 +102,7 @@ def userpage(username):
     user_freq_record = path_prefix + 'static/frequency/' + 'frequency_%s.pickle' % (username)
 
     if request.method == 'POST':  # when we submit a form
-        content = escape(request.form['content'])
+        content = request.form['content']
         f = WordFreq(content)
         lst = f.get_freq()
         return render_template('userpage_post.html',username=username,lst = lst, yml=Yaml.yml)

app/wordfreqCMD.py

@@ -39,7 +39,7 @@ def file2str(fname):#文件转字符
 
 
 def remove_punctuation(s): # 这里是s是形参 (parameter)。函数被调用时才给s赋值。
-    special_characters = '\_©~<=>+-/[]*&$%^@.,?!:;#()"“”—‘’{}|' # 把里面的字符都去掉
+    special_characters = '_©~=+[]*&$%^@.,?!:;#()"“”—‘’' # 把里面的字符都去掉
     for c in special_characters:
         s = s.replace(c, ' ') # 防止出现把 apple,apple 移掉逗号后变成 appleapple 情况
     s = s.replace('--', ' ')

build.sh

@@ -1,6 +1,6 @@
 #!/bin/sh
 
-DEPLOYMENT_DIR=/home/lanhui/englishpal2/EnglishPal
+DEPLOYMENT_DIR=/home/lanhui/EnglishPal
 cd $DEPLOYMENT_DIR
 
 # Stop service
@@ -11,7 +11,7 @@ sudo docker rm EnglishPal
 sudo docker build -t englishpal .
 
 # Run the application
-sudo docker run --restart=always -d --name EnglishPal -p 90:80 -v ${DEPLOYMENT_DIR}/app/static/frequency:/app/static/frequency -v ${DEPLOYMENT_DIR}/app/static/:/app/static/ -t englishpal  # for permanently saving data
+sudo docker run -d --name EnglishPal -p 90:80 -v ${DEPLOYMENT_DIR}/app/static/frequency:/app/static/frequency -v ${DEPLOYMENT_DIR}/app/static/:/app/static/ -t englishpal  # for permanently saving data
 
 # Save space.  Run it after sudo docker run
 sudo docker system prune -a -f