铁路4的动作邮件

Question

为了创建一个联系人表单，然后创建一个发送给站点管理员的邮件程序，我一直在遵循一个相当古老的教程。我遇到了一些问题，我想，因为我在Rails4.1上。1而且教程太旧了！我想知道是否有人能给我一些指导——我正在学习rails。我当前收到的错误是MessagesController为主题消息创建未定义的方法“subject”中的NoMethodError。主题

这是我的密码

消息控制器：

class MessagesController < ApplicationController
  before_action :set_message, only: [:show, :edit, :update, :destroy]

  # GET /messages
  # GET /messages.json
  def index
    @messages = Message.all
  end

  # GET /messages/1
  # GET /messages/1.json
  def show
  end

  # GET /messages/new
  def new
    @message = Message.new
  end

  # GET /messages/1/edit
  def edit
  end

  # POST /messages
  # POST /messages.json
  def create
    @message = Message.new(message_params)

respond_to do |format|
  if @message.save
    ContactMailer.deliver_message(@message)
    flash.now[:notice] = 'Thank you for your message!'
    format.html { redirect_to root_path }
    format.json { render :show, status: :created, location: @message }
  else
    format.html { render :new }
    format.json { render json: @message.errors, status: :unprocessable_entity }
  end
end

终止

#PATCH/PUT/messages/1#PATCH/PUT/messages/1。json def update respond|u to do | format | if@message。更新（消息参数）格式。html{重定向到@message，注意：'消息已成功更新。}格式。json{render:show，status:：ok，location:@message}else格式。html{呈现：编辑}格式。json{render json:@message.errors，status:：unprocessable_entity}end

#删除/messages/1#删除/messages/1。json def destroy@message。销毁响应|格式|格式。html{redirect_to messages_url，注意：'Message已成功销毁。}格式。json{head:no_content}end

private#使用回调在操作之间共享公共设置或约束。def set_message@message=message。查找（参数[：id]）结束

# Never trust parameters from the scary internet, only allow the white list through.
def message_params
  params.require(:message).permit(:name, :email, :company, :phone, :subject, :body)
end

终止

development.rb：

Rails.application.configure do
  # Settings specified here will take precedence over those in config/application.rb.

  # In the development environment your application's code is reloaded on
  # every request. This slows down response time but is perfect for development
  # since you don't have to restart the web server when you make code changes.
  config.cache_classes = false

  # Do not eager load code on boot.
  config.eager_load = false

  # Show full error reports and disable caching.
  config.consider_all_requests_local       = true
  config.action_controller.perform_caching = false

  # Don't care if the mailer can't send.
  config.action_mailer.raise_delivery_errors = true

  config.action_mailer.delivery_method = :smtp
  config.action_mailer.smtp_settings = {
    address: "smpt.gmail.com",
    port: "587",
    domain: "gmail.com",
    authentication: "plain",
    enable_starttls_auto: true,
    user_name: 'your_email_username',
    password: 'your_email_password'
  }

  CONTACT_RECIPIENT = '[e-mail]'
  # Print deprecation notices to the Rails logger.
  config.active_support.deprecation = :log

  # Raise an error on page load if there are pending migrations.
  config.active_record.migration_error = :page_load

  # Debug mode disables concatenation and preprocessing of assets.
  # This option may cause significant delays in view rendering with a large
  # number of complex assets.
  config.assets.debug = true

  # Adds additional error checking when serving assets at runtime.
  # Checks for improperly declared sprockets dependencies.
  # Raises helpful error messages.
  config.assets.raise_runtime_errors = true

  # Raises error for missing translations
  # config.action_view.raise_on_missing_translations = true

  # Required for Devise gem
  config.action_mailer.default_url_options = { :host => 'localhost:3000' }

end

class ContactMailer < ActionMailer::Base
  default from: "[e-mail]"

  # Subject can be set in your I18n file at config/locales/en.yml
  # with the following lookup:
  #
  #   en.contact_mailer.message.subject
  #

    def message(message)
    subject    message.subject
    body       :message => message
    recipients CONTACT_RECIPIENT
    from       message.email
    sent_on    Time.now
    end

  end
end

消息html。雇员再培训局

Email from your web site

From: <%= @message.name %>

Company: <%= @message.company %>

Phone: <%= @message.phone %>

Message: <%= @message.body %>

信息/_形式。html。雇员再培训局

<%= semantic_form_for @message do |f| %>
  <%= f.inputs do %>
    <%= f.input :name %>
    <%= f.input :company %>
    <%= f.input :phone %>
    <%= f.input :email %>
    <%= f.input :subject %>
    <%= f.input :body %>
  <% end %>

  <%= f.actions do %>
    <%= f.action :submit, :as => :input %>
  <% end %>
<% end %>

数据库模式文件的相关部分

 create_table "messages", force: true do |t|
    t.string   "name"
    t.string   "company"
    t.string   "phone"
    t.string   "email"
    t.string   "subject"
    t.text     "body"
    t.datetime "created_at"
    t.datetime "updated_at"
  end

Answer 1

def deliver_message(message)
  subject    message.subject
  body       :message => message
  recipients CONTACT_RECIPIENT
  from       message.email
  sent_on    Time.now
end

应该是

def deliver_message(message)
  @message = message
  mail(subject: message.subject, body: message.body, recipients: CONTACT_RECIPIENT, from: message.email, sent_on: Time.now)
 end