Skip to content

Commit 7e617e2

Browse files
spevansianpartridge
authored andcommitted
JSONEncoder: Rename data coding enum values to align with SE-0167 (#1219)
- DataEncodingStrategy.base64Encode -> .base64 - DataDecodingStrategy.base64Decode -> .base64
1 parent a7c033b commit 7e617e2

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

Foundation/JSONEncoder.swift

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ open class JSONEncoder {
6565
/// The strategy to use for encoding `Data` values.
6666
public enum DataEncodingStrategy {
6767
/// Encoded the `Data` as a Base64-encoded string. This is the default strategy.
68-
case base64Encode
68+
case base64
6969

7070
/// Encode the `Data` as a custom value encoded by the given closure.
7171
///
@@ -88,8 +88,8 @@ open class JSONEncoder {
8888
/// The strategy to use in encoding dates. Defaults to `.deferredToDate`.
8989
open var dateEncodingStrategy: DateEncodingStrategy = .deferredToDate
9090

91-
/// The strategy to use in encoding binary data. Defaults to `.base64Encode`.
92-
open var dataEncodingStrategy: DataEncodingStrategy = .base64Encode
91+
/// The strategy to use in encoding binary data. Defaults to `.base64`.
92+
open var dataEncodingStrategy: DataEncodingStrategy = .base64
9393

9494
/// The strategy to use in encoding non-conforming numbers. Defaults to `.throw`.
9595
open var nonConformingFloatEncodingStrategy: NonConformingFloatEncodingStrategy = .throw
@@ -672,7 +672,7 @@ extension _JSONEncoder {
672672

673673
fileprivate func box(_ data: Data) throws -> NSObject {
674674
switch self.options.dataEncodingStrategy {
675-
case .base64Encode:
675+
case .base64:
676676
return NSString(string: data.base64EncodedString())
677677

678678
case .custom(let closure):
@@ -827,7 +827,7 @@ open class JSONDecoder {
827827
/// The strategy to use for decoding `Data` values.
828828
public enum DataDecodingStrategy {
829829
/// Decode the `Data` from a Base64-encoded string. This is the default strategy.
830-
case base64Decode
830+
case base64
831831

832832
/// Decode the `Data` as a custom value decoded by the given closure.
833833
case custom((_ decoder: Decoder) throws -> Data)
@@ -845,8 +845,8 @@ open class JSONDecoder {
845845
/// The strategy to use in decoding dates. Defaults to `.deferredToDate`.
846846
open var dateDecodingStrategy: DateDecodingStrategy = .deferredToDate
847847

848-
/// The strategy to use in decoding binary data. Defaults to `.base64Decode`.
849-
open var dataDecodingStrategy: DataDecodingStrategy = .base64Decode
848+
/// The strategy to use in decoding binary data. Defaults to `.base64`.
849+
open var dataDecodingStrategy: DataDecodingStrategy = .base64
850850

851851
/// The strategy to use in decoding non-conforming numbers. Defaults to `.throw`.
852852
open var nonConformingFloatDecodingStrategy: NonConformingFloatDecodingStrategy = .throw
@@ -2042,7 +2042,7 @@ extension _JSONDecoder {
20422042
guard !(value is NSNull) else { return nil }
20432043

20442044
switch self.options.dataDecodingStrategy {
2045-
case .base64Decode:
2045+
case .base64:
20462046
guard let string = value as? String else {
20472047
throw DecodingError._typeMismatch(at: self.codingPath, expectation: type, reality: value)
20482048
}

TestFoundation/TestJSONEncoder.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -446,8 +446,8 @@ class TestJSONEncoder : XCTestCase {
446446
outputFormatting: JSONEncoder.OutputFormatting = [],
447447
dateEncodingStrategy: JSONEncoder.DateEncodingStrategy = .deferredToDate,
448448
dateDecodingStrategy: JSONDecoder.DateDecodingStrategy = .deferredToDate,
449-
dataEncodingStrategy: JSONEncoder.DataEncodingStrategy = .base64Encode,
450-
dataDecodingStrategy: JSONDecoder.DataDecodingStrategy = .base64Decode,
449+
dataEncodingStrategy: JSONEncoder.DataEncodingStrategy = .base64,
450+
dataDecodingStrategy: JSONDecoder.DataDecodingStrategy = .base64,
451451
nonConformingFloatEncodingStrategy: JSONEncoder.NonConformingFloatEncodingStrategy = .throw,
452452
nonConformingFloatDecodingStrategy: JSONDecoder.NonConformingFloatDecodingStrategy = .throw) where T : Codable, T : Equatable {
453453
var payload: Data! = nil

0 commit comments

Comments
 (0)