Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

從 依資本額排序 切換至 依股價排序 會導致誤植公司名與連結 - 股市總覽 #586

Open
SoftwareSing opened this issue Feb 9, 2019 · 1 comment
Labels

Comments

@SoftwareSing
Copy link
Member

SoftwareSing commented Feb 9, 2019

Chrome 版本 72.0.3626.96
登入狀態不影響

股市總覽 頁面
依資本額排序 切換至 依股價排序

部分 公司名稱 與 連結 會因此錯置位置
其餘資訊、圖片不受影響

已確認客戶端內dbCompanies內的資訊沒有錯誤
誤植的公司名稱不在客戶端的dbCompanies


僅確認在 2019/02/10 03:22 (GMT+8) 的正式伺服器可以觸發
公司ID7wbR4aRkdkQTNstax(初音) 之卡面被誤植 公司ID5EM3XbXQFxCNZWn8L(英梨梨) 的名稱與連結
公司IDmgZDFpDCwearEr3cJ(惠惠) 之卡面被誤植 公司ID7wbR4aRkdkQTNstax(初音) 的名稱與連結
image

5EM3XbXQFxCNZWn8L(英梨梨)並沒有出現在客戶端的dbCompanies


使用最接近的備份 2019-02-09-salaryBefore.tar.gz 無法在測試伺服器重現BUG

使用備份 2019-02-10-seasonAfter.tar.gz 可以在測試伺服器重現BUG


相關issue:
#68


相關code:
https://github.com/mrbigmouth/acgn-stock/blob/5ff02980c6b573b4cda647471eec7de38fa7a7a8/client/company/companyList.html#L111


我想companyLink應該是沒問題的,只是丟進去的id為什麼錯了...............

@frozenmouse
Copy link
Contributor

frozenmouse commented Feb 10, 2019

直覺應該是 ajax 造成的問題
因為公司名稱是用 ajax 方式和 server 要的
在 template re-render 的時候沒有觸發到這部分,所以沒更新

這和 #491 是同樣問題,只是當時只有針對 userLink 修正
解法是用 autorun 來監視 template 的 data context 變動來觸發更新就可以了

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants