達人🐉☄️💫
@c910335
Wed, Aug 2, 2017 4:10 PM
#Crystal #Amber
redirect_to and render should be able to halt the execution, even...
這個真的很困擾
可是看他 code 又覺得很難實作
只好開個 issue
掰噗~
@baipu
說
Wed, Aug 2, 2017 4:10 PM
拍拍
機器狼🔜茶會首次出毛!
@KMN_BOT
說
Wed, Aug 2, 2017 4:12 PM
機器狼也這麼覺得呢
達人🐉☄️💫
@c910335
Wed, Aug 2, 2017 4:45 PM
l• ܫ•) Davyキュルッ
www
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 4:48 PM
看起來只要把
amber-crystal/amber
裡面的東西抽回 controller 然後判斷一下就可以了
還可以順便解決
amber-crystal/amber
的問題
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 4:55 PM
這裡的問題應該是因為 crystal 沒辦法動態呼叫函式所以必須要用 macro 包裝
不確定不同 class 內的 macro 被呼叫會怎樣,如果可行的話應該是把 macro 整個搬過去就解決惹
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 4:59 PM
啊 不能呼叫 我想想看哦
達人🐉☄️💫
@c910335
Wed, Aug 2, 2017 5:16 PM
他在 Gitter 問我為什麼 render 也需要
可是我沒想到什麼好的理由
只是單純覺得也該如此
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 5:22 PM
其實應該用 return
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 5:23 PM
額 我是用在 Rails 的邏輯去想啦,可是我覺得 amber 可以有別的邏輯
但是我覺得 halt 不錯,要是 halt 還能帶一個 content 就更好了
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 5:24 PM
其實 render 跟 redirect_to 應該都不需要才對
他們應該去改 response 物件,但是什麼時候斷掉應該要可以另外控制
達人🐉☄️💫
@c910335
Wed, Aug 2, 2017 5:28 PM
不過 redirect_to 這方法名字就會讓人覺得該在這裡結束了吧
達人🐉☄️💫
@c910335
Wed, Aug 2, 2017 5:29 PM
render 的話可能是 rails 邏輯吧QQ
達人🐉☄️💫
@c910335
Wed, Aug 2, 2017 5:31 PM
不過達人睏了要睡覺了 睡醒再回來討論
l• ܫ•) Davyキュルッ
@david50407
Wed, Aug 2, 2017 5:36 PM
看了一下整個邏輯很不一樣啊 所以我也來想一下XD
載入新的回覆
redirect_to and render should be able to halt the execution, even...
這個真的很困擾
可是看他 code 又覺得很難實作
只好開個 issue
amber-crystal/amber
裡面的東西抽回 controller 然後判斷一下就可以了
還可以順便解決 amber-crystal/amber 的問題
不確定不同 class 內的 macro 被呼叫會怎樣,如果可行的話應該是把 macro 整個搬過去就解決惹
可是我沒想到什麼好的理由
只是單純覺得也該如此
但是我覺得 halt 不錯,要是 halt 還能帶一個 content 就更好了
他們應該去改 response 物件,但是什麼時候斷掉應該要可以另外控制