hyperstack-org/hyperstack

trying to destroy an item that is not included in the default scope fails

Open

#260 opened on 2019年10月3日

GitHub で見る
 (1 comment) (0 reactions) (0 assignees)JavaScript (531 stars) (43 forks)batch import
good first issue

説明

If you have a default_scope (other than just all) then some records are not included by a simple query. However you can get you can override the default scope using unscoped. This means you can get the id of records that are not in the default scope.

If you try to destroy one of those records, it doesn't work.

Fix would be on about line 590 of isomorphic_base.rb in the server side destroy_record method. Just change model.find(id) to model.unscoped.find(id)

コントリビューターガイド

技術スタック
rubyrails
領域
backend
Issue 種別
bug
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Ruby on RailsActiveRecord
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
85
調査方針
Look at the `destroy record` method in `isomorphic base.rb` around line 590. The fix is to change `model.find(id)` to `model.unscoped.find(id)`. Ensure the change works with all model scopes and add a test case for destroying records excluded by default scopes.