Add inverse_of to Promotion#codes association

This helps fix an issue when creating a new promotion with a single code. Without the inverse_of parameter, our associated codes won't correctly infer that they're associated to a new promotion and will generate undefined method for nil errors. Co-authored-by: Alistair Norman <[email protected]> (cherry picked from commit 2cb9ac1)
solidusio · Dec 6, 2024 · a1038fd · a1038fd
1 parent bfdf4bb
commit a1038fd
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/promotions/app/models/solidus_promotions/promotion.rb b/promotions/app/models/solidus_promotions/promotion.rb
@@ -9,7 +9,7 @@ class Promotion < Spree::Base
     belongs_to :original_promotion, class_name: "Spree::Promotion", optional: true
     has_many :benefits, class_name: "SolidusPromotions::Benefit", dependent: :destroy
     has_many :conditions, through: :benefits
-    has_many :codes, class_name: "SolidusPromotions::PromotionCode", dependent: :destroy
+    has_many :codes, class_name: "SolidusPromotions::PromotionCode", dependent: :destroy, inverse_of: :promotion
     has_many :code_batches, class_name: "SolidusPromotions::PromotionCodeBatch", dependent: :destroy
     has_many :order_promotions, class_name: "SolidusPromotions::OrderPromotion", dependent: :destroy