質問編集履歴

ソースコードとエラーの内容が異なっていたため、修正致しました。(ActionActivationControllerのeditアクションの部分)

2021/02/12 12:31

投稿

gussan-me

スコア7

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -52,6 +52,190 @@
     if user && !user.activated? && user.authenticated?(:activation, params[:id])
+      user.activate
+      log_in user
+      flash[:success] = "Account activated!"
+      redirect_to user
+    else
+      flash[:danger] = "Invalid activation link"
+      redirect_to root_url
+    end
+  end
+end
+```
+```
+#AccountActivationsControllerに関係するモデルです。
+#authenticated?メソッドとactivateメソッドの情報が必要だと思い、掲載させていただきました。
+class User < ApplicationRecord
+  attr_accessor :remember_token,:activation_token
+  before_save   :downcase_email
+  before_create :create_activation_digest
+  validates :name,  presence: true, length: { maximum: 50 }
+  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(.[a-z\d\-]+)*.[a-z]+\z/i
+  validates :email, presence: true, length: { maximum: 255 },
+                    format: { with: VALID_EMAIL_REGEX },
+                    uniqueness: true
+  has_secure_password
+  validates :password, presence: true, length: { minimum: 6 },allow_nil: true
+  # 渡された文字列のハッシュ値を返す
+  def User.digest(string)
+    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
+                                                  BCrypt::Engine.cost
+    BCrypt::Password.create(string, cost: cost)
+  end
+  # ランダムなトークンを返す
+  def User.new_token
+    SecureRandom.urlsafe_base64
+  end
+  # 永続セッションのためにユーザーをデータベースに記憶する
+  def remember
+    self.remember_token = User.new_token
+    update_attribute(:remember_digest, User.digest(remember_token))
+  end
+  # 渡されたトークンがダイジェストと一致したらtrueを返す
+  def authenticated?(attribute, token)
+    digest = send("#{attribute}_digest")
+    return false if digest.nil?
+    BCrypt::Password.new(digest).is_password?(token)
+  end
+  def forget
+    update_attribute(:remember_digest, nil)
+  end
+  def activate
+    update_attribute(:activated,    true)
+    update_attribute(:activated_at, Time.zone.now)
+  end
+  # 有効化用のメールを送信する
+  def send_activation_email
+    UserMailer.account_activation(self).deliver_now
+  end
+  private
+    # メールアドレスをすべて小文字にする
+    def downcase_email
+     self.email = email.downcase
+    end
+    # 有効化トークンとダイジェストを作成および代入する
+    def create_activation_digest
+     self.activation_token  = User.new_token
+     self.activation_digest = User.digest(activation_token)
+    end
+end
+```
+###試したこと
+editアクションのif文の１行目
+`if user && !user.activated? && user.authenticated?(:activation, params[:id])`
+が強制的に成功するように書き換えて見た。
+( `if user == user`にして見た。  )
+```
+class AccountActivationsController < ApplicationController
+  def edit
+    user = User.find_by(email: params[:email])
+    if user == user
       user.update_attribute(:activated,    true)
       user.update_attribute(:activated_at, Time.zone.now)
@@ -76,192 +260,6 @@
 ```
-```
-#AccountActivationsControllerに関係するモデルです。
-#authenticated?メソッドとactivateメソッドの情報が必要だと思い、掲載させていただきました。
-class User < ApplicationRecord
-  attr_accessor :remember_token,:activation_token
-  before_save   :downcase_email
-  before_create :create_activation_digest
-  validates :name,  presence: true, length: { maximum: 50 }
-  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(.[a-z\d\-]+)*.[a-z]+\z/i
-  validates :email, presence: true, length: { maximum: 255 },
-                    format: { with: VALID_EMAIL_REGEX },
-                    uniqueness: true
-  has_secure_password
-  validates :password, presence: true, length: { minimum: 6 },allow_nil: true
-  # 渡された文字列のハッシュ値を返す
-  def User.digest(string)
-    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
-                                                  BCrypt::Engine.cost
-    BCrypt::Password.create(string, cost: cost)
-  end
-  # ランダムなトークンを返す
-  def User.new_token
-    SecureRandom.urlsafe_base64
-  end
-  # 永続セッションのためにユーザーをデータベースに記憶する
-  def remember
-    self.remember_token = User.new_token
-    update_attribute(:remember_digest, User.digest(remember_token))
-  end
-  # 渡されたトークンがダイジェストと一致したらtrueを返す
-  def authenticated?(attribute, token)
-    digest = send("#{attribute}_digest")
-    return false if digest.nil?
-    BCrypt::Password.new(digest).is_password?(token)
-  end
-  def forget
-    update_attribute(:remember_digest, nil)
-  end
-  def activate
-    update_attribute(:activated,    true)
-    update_attribute(:activated_at, Time.zone.now)
-  end
-  # 有効化用のメールを送信する
-  def send_activation_email
-    UserMailer.account_activation(self).deliver_now
-  end
-  private
-    # メールアドレスをすべて小文字にする
-    def downcase_email
-     self.email = email.downcase
-    end
-    # 有効化トークンとダイジェストを作成および代入する
-    def create_activation_digest
-     self.activation_token  = User.new_token
-     self.activation_digest = User.digest(activation_token)
-    end
-end
-```
-###試したこと
-editアクションのif文の１行目
-`if user && !user.activated? && user.authenticated?(:activation, params[:id])`
-が強制的に成功するように書き換えて見た。
-( `if user == user`にして見た。  )
-```
-class AccountActivationsController < ApplicationController
-  def edit
-    user = User.find_by(email: params[:email])
-    if user == user
-      user.update_attribute(:activated,    true)
-      user.update_attribute(:activated_at, Time.zone.now)
-      log_in user
-      flash[:success] = "Account activated!"
-      redirect_to user
-    else
-      flash[:danger] = "Invalid activation link"
-      redirect_to root_url
-    end
-  end
-end
-```
 こうした結果、下の画像のようなエラーが出ました。
 userに原因があるのかなと考えているのですが、どういった原因があるのか検討がつきません。。