1. 24 12月, 2013 6 次提交
    • C
      Tidy up fix for PG extensions quoting · 9e1740af
      Carlos Antonio da Silva 提交于
      Always pass in the column for quote_bound_value and quote using it in
      case it exists there.
      9e1740af
    • T
      Serialize postgres' hstore, json and array types correctly in AR update methods. · 73bba4c1
      Tadas Tamosauskas 提交于
      Fixes #12261. Closes #12395.
      
      Conflicts:
      	activerecord/CHANGELOG.md
      	activerecord/test/cases/adapters/postgresql/array_test.rb
      	activerecord/test/cases/adapters/postgresql/json_test.rb
      73bba4c1
    • C
      Do not consider PG array columns as number or text columns · 56510091
      Carlos Antonio da Silva 提交于
      The code uses these checks in several places to know what to do with a
      particular column, for instance AR attribute query methods has a branch
      like this:
      
          if column.number?
            !value.zero?
          end
      
      This should never be true for array columns, since it would be the same
      as running [].zero?, which results in a NoMethodError exception.
      
      Fixing this by ensuring that array columns in PostgreSQL never return
      true for number?/text? checks.
      
      Since most of the array support was based on the postgres_ext lib, it's
      worth noting it does the same thing for numeric array columns too:
      
      https://github.com/dockyard/postgres_ext/blob/v1.0.0/lib/postgres_ext/active_record/connection_adapters/postgres_adapter.rb#L72
      
      This extended the same logic for text columns to ensure consistency.
      56510091
    • C
      Move the null mime type to request.format · 618d5317
      Carlos Antonio da Silva 提交于
      TLDR: always return an object that responds to the query methods from
      request.format, and do not touch Mime::Type[] lookup to avoid bugs.
      
      ---
      
      Long version:
      
      The initial issue was about being able to do checks like
      request.format.html? for request with an unknown format, where
      request.format would be nil.
      
      This is where the issue came from at first in #7837 and #8085
      (merged in cba05887), but the
      implementation went down the path of adding this to the mime type
      lookup logic.
      
      This unfortunately introduced subtle bugs, for instance in the merged
      commit a test related to send_file had to be changed to accomodate the
      introduction of the NullType.
      
      Later another bug was found in #13064, related to the content-type being
      shown as #<Mime::NullType:...> for templates with localized extensions
      but no format included. This one was fixed in #13133, merged in
      43962d6e.
      
      Besides that, custom handlers were not receiving the proper template
      formats anymore when passing through the rendering process, because of
      the NullType addition. That was found while migrating an application
      from 3.2 to 4.0 that uses the Markerb gem (a custom handler that
      generates both text and html emails from a markdown template).
      
      ---
      
      This changes the implementation moving away from returning this null
      object from the mime lookup, and still fixes the initial issue where
      request.format.zomg? would raise an exception for unknown formats due to
      request.format being nil.
      618d5317
    • G
      Merge pull request #13462 from JuanitoFatas/getting-started · ed334d9a
      Godfrey Chan 提交于
      [ci skip] a destroy action in controller, not delete [getting_stated.md].
      ed334d9a
    • J
  2. 23 12月, 2013 12 次提交
  3. 22 12月, 2013 10 次提交
  4. 21 12月, 2013 9 次提交
  5. 20 12月, 2013 3 次提交