- 03 12月, 2013 1 次提交
-
-
由 Godfrey Chan 提交于
Also added the missing CHANGELOG entry for #12183 @ 80e75520 and 4d02296c.
-
- 27 11月, 2013 5 次提交
-
-
由 Godfrey Chan 提交于
-
由 Godfrey Chan 提交于
Got all the tests passing again. Support for `encode_json` has been removed (and consequently the ability to encode `BigDecimal`s as numbers, as mentioned in the previous commit). Install the `activesupport-json_encoder` gem to get it back.
-
由 Godfrey Chan 提交于
This is because the new encoder will no longer support encode_json. Therefore our only choice is to return `to_i` or `to_s` in `BigDecimal#as_json`. Since casting a BigDecimal to an integer is most likely a lossy operation, we chose to encode it as a string. Support for encoding BigDecimal as a string will return via the `activesupport-json_encoder` gem.
-
由 Godfrey Chan 提交于
-
由 Godfrey Chan 提交于
-
- 23 11月, 2013 1 次提交
-
-
由 Godfrey Chan 提交于
should also call #as_json on the children without options (instead of nil)
-
- 15 11月, 2013 1 次提交
-
-
由 Godfrey Chan 提交于
Previously, calling `::JSON.{generate,dump}` sometimes causes unexpected failures such as intridea/multi_json#86. `::JSON.{generate,dump}` now bypasses the ActiveSupport JSON encoder completely and yields the same result with or without ActiveSupport. This means that it will **not** call `as_json` and will ignore any options that the JSON gem does not natively understand. To invoke ActiveSupport's JSON encoder instead, use `obj.to_json(options)` or `ActiveSupport::JSON.encode(obj, options)`.
-
- 07 11月, 2013 2 次提交
-
-
由 Ryan Glover 提交于
-
由 Godfrey Chan 提交于
See [1] for why this is not a good idea. As part of this refactor, circular reference protection in as_json has been removed and the corresponding error class has been deprecated. As discussed with @jeremy, circular reference error is considered programmer errors and protecting against it is out of scope for the encoder. This is again based on the excellent work by @sergiocampama in #11728. [1]: https://github.com/intridea/multi_json/pull/138#issuecomment-24468223
-
- 06 11月, 2013 2 次提交
-
-
由 Godfrey Chan 提交于
JSON.{dump,generate} offered by the JSON gem is not compatiable with Rails at the moment and can cause a lot of subtle bugs when passed certain data structures. This changed all direct usage of the JSON gem in internal Rails code to always go through AS::JSON.{decode,encode}. We also shouldn't be implementing `to_json` most of the time, and these occurances are replaced with an equivilent `as_json` implementation to avoid problems down the road. See [1] for all the juicy details. [1]: intridea/multi_json#138 (comment)
-
由 Godfrey Chan 提交于
These methods now takes the same options as Hash#as_json, for example: struct = Struct.new(:foo, :bar).new struct.foo = "hello" struct.bar = "world" json = struct.as_json(only: [:foo]) # => {foo: "hello"} This is extracted from PR #11728 from @sergiocampama, see also the discussion in #11460.
-
- 13 9月, 2013 1 次提交
-
-
由 Godfrey Chan 提交于
TL;DR The primary driver is to remove autoload surprise. This is related to #12106. (The root cause for that ticket is that json/add defines Regexp#to_json among others, but here I'll reproduce the problem without json/add.) Before: >> require 'active_support/core_ext/to_json' => true >> //.as_json NoMethodError: undefined method `as_json' for //:Regexp from (irb):3 from /Users/godfrey/.rvm/rubies/ruby-2.0.0-p195/bin/irb:16:in `<main>' >> //.to_json => "\"(?-mix:)\"" >> //.as_json => "(?-mix:)" After: >> require 'active_support/core_ext/to_json' => true >> //.as_json => "(?-mix:)" This is because ActiveSupport::JSON is autoloaded the first time Object#to_json is called, which causes additional core extentions (previously defined in active_support/json/encoding.rb) to be loaded. When someone require 'active_support/core_ext', the expectation is that it would add certain methods to the core classes NOW. The previous behaviour causes additional methods to be loaded the first time you call `to_json`, which could cause nasty surprises and other unplesant side-effects. This change moves all core extensions in to core_ext/json. AS::JSON is still autoloaded on first #to_json call, but since it nolonger include the core extensions, it should address the aforementioned bug. *Requiring core_ext/object/to_json now causes a deprecation warnning*
-
- 01 6月, 2013 1 次提交
-
-
由 kennyj 提交于
-
- 26 5月, 2013 1 次提交
-
-
由 Vipul A M 提交于
-
- 09 5月, 2013 1 次提交
-
-
由 Mario Caropreso 提交于
U+2028 and U+2029 are allowed inside strings in JSON (as all literal Unicode characters) but JavaScript defines them as newline seperators. Because no literal newlines are allowed in a string, this causes a ParseError in the browser. We work around this issue by replacing them with the escaped version. The resulting JSON is still valid and can be parsed in the browser. This commit has been coauthored with Viktor Kelemen @yikulju
-
- 04 5月, 2013 1 次提交
-
-
由 Ryan Davis 提交于
-
- 28 12月, 2012 1 次提交
-
-
由 Yves Senn 提交于
As reported (https://github.com/rails/rails/pull/8185#issuecomment-11702226) this test relied on the order a hash was serialized. Comparing the parsed hash makes the test no longer order dependent.
-
- 15 12月, 2012 1 次提交
-
-
由 Brett Carter 提交于
The encoding scheme (e.g.
☠ -> "\u2620") was broken for characters not in the Basic Multilingual Plane. It is possible to escape them for json using the weird encoding scheme of a twelve-character sequence representing the UTF-16 surrogate pair (e.g. '𠜎' -> "\u270e\u263a") but this wasn't properly handled in the escaping code. Since raw UTF-8 is allowed in json, it was decided to simply pass through the raw bytes rather than attempt to escape them.
-
- 12 11月, 2012 1 次提交
-
-
由 Yves Senn 提交于
Setting options in a custom `#as_json` method had side effects. Modifications of the `options` hash leaked outside and influenced the conversion of other objects contained in the hash.
-
- 08 8月, 2012 1 次提交
-
-
由 Erich Menge 提交于
Reason: ActiveSupport::JSON::Variable is not used anymore internally. It was deprecated in 3-2-stable but we reverted all the deprecation for point releases. See #6536 and #6546. Conflicts: activesupport/lib/active_support/json/variable.rb
-
- 31 5月, 2012 1 次提交
-
-
由 Erich Menge 提交于
-
- 30 5月, 2012 1 次提交
-
-
由 Erich Menge 提交于
-
- 02 5月, 2012 1 次提交
-
-
由 David FRANCOIS 提交于
-
- 01 5月, 2012 1 次提交
-
-
由 Sebi Burkhard 提交于
-
- 06 1月, 2012 1 次提交
-
-
由 Aaron Patterson 提交于
-
- 02 1月, 2012 1 次提交
-
-
由 Sergey Nartimov 提交于
-
- 25 12月, 2011 1 次提交
-
-
由 Sergey Nartimov 提交于
-
- 21 12月, 2011 1 次提交
-
-
由 José Valim 提交于
There are a bunch of other implicit branches that adds 1.8.x specific code that still needs to be removed. Pull requests for those cases are welcome.
-
- 15 8月, 2011 1 次提交
-
-
由 Sebi Burkhard 提交于
-
- 02 4月, 2011 1 次提交
-
-
由 John Firebaugh 提交于
-
- 18 1月, 2011 1 次提交
-
-
由 Aaron Patterson 提交于
-
- 15 1月, 2011 1 次提交
-
-
由 Alexey Nayden 提交于
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>
-
- 12 1月, 2011 1 次提交
-
-
由 Josh Kalderimis 提交于
Signed-off-by: NSantiago Pastorino <santiago@wyeworks.com>
-
- 08 9月, 2010 1 次提交
-
-
由 Jakub Suder 提交于
- as_json in ActiveModel should return a hash and handle :only/:except/:methods options - Array and Hash should call as_json on their elements - json methods should not modify options argument [#5374 state:committed] Signed-off-by: NJeremy Kemper <jeremy@bitsweat.net>
-
- 24 7月, 2010 1 次提交
-
-
由 Santiago Pastorino 提交于
Signed-off-by: NJosé Valim <jose.valim@gmail.com>
-
- 02 7月, 2010 1 次提交
-
-
由 Santiago Pastorino 提交于
[#4979 state:committed] Signed-off-by: NJosé Valim <jose.valim@gmail.com>
-
- 04 5月, 2010 1 次提交
-
-
由 Xavier Noria 提交于
BigDecimal#as_json does not really specify the F format, it delegates that to whatever BigDecimal#to_s default format is, do the same in its test
-
- 29 4月, 2010 1 次提交
-
-
由 Cezary Baginski 提交于
Signed-off-by: NJeremy Kemper <jeremy@bitsweat.net>
-
- 27 4月, 2010 1 次提交
-
-
由 Jeremy Kemper 提交于
JSON: encode objects that don't have a native JSON representation using to_hash, if available, instead of instance_values (the old fallback) or to_s (other encoders' default). Encode BigDecimal and Regexp encode as strings to conform with other encoders. Try to transcode non-UTF-8 strings.
-