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

hideコマンドを修正 #34

Merged
merged 2 commits into from
Apr 19, 2020
Merged

hideコマンドを修正 #34

merged 2 commits into from
Apr 19, 2020

Conversation

cahlchang
Copy link
Owner

https://github.com/cahlchang/CoCNonKP/blob/master/lambda_function.py#L637
ここでunquoteからunquote_plusに変わっていて正規表現文字列の修正が必要だった。
通常lambdaはコマンドの半角スペースを+として解釈するが
unquote_plusではさらに+をスペースに変換する。
もともとは+を期待した正規表現だったため解釈が出来なかった。

(余談)
もともとunquote_plusを使わなかった理由はコマンド上で利用される「+」まで
半角スペースに変更されるケースがあった記憶。(記憶違いかもしれない)
・例
/cc hide 心理学+10
→unquote message
hide+心理学+10
→unquote_plus message
hide 心理学 10

ただ、今確認してみると取りたい文字列で取れている様子。
→current message
hide 心理学+10

どこかで対応してくれたのか、とりあえず良さそうなのでこのままで。

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

Successfully merging this pull request may close these issues.

1 participant