Skip to content

Commit 198bf43

Browse files
committed
Limit Site Post fields to siteID and password
1 parent c99b136 commit 198bf43

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

Networking/Networking/Remote/SitePostsRemote.swift

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ public class SitePostsRemote: Remote {
1717
///
1818
public func loadSitePost(for siteID: Int64, postID: Int64, completion: @escaping (Post?, Error?) -> Void) {
1919
let path = String(format: "/sites/%d/posts/%d", siteID, postID)
20-
let request = DotcomRequest(wordpressApiVersion: .mark1_1, method: .get, path: path)
20+
let parameters = ["fields": "site_ID,password"]
21+
let request = DotcomRequest(wordpressApiVersion: .mark1_1, method: .get, path: path, parameters: parameters)
2122
let mapper = PostMapper()
2223

2324
enqueue(request, mapper: mapper, completion: completion)
@@ -34,7 +35,9 @@ public class SitePostsRemote: Remote {
3435
///
3536
public func updateSitePost(for siteID: Int64, postID: Int64, post: Post, completion: @escaping (Post?, Error?) -> Void) {
3637
do {
37-
let parameters = try post.toDictionary()
38+
var parameters = try post.toDictionary()
39+
let parametersFields = ["fields": "site_ID,password"]
40+
parameters.merge(parametersFields) { (current, _) in current }
3841
let path = String(format: "/sites/%d/posts/%d", siteID, postID)
3942
let request = DotcomRequest(wordpressApiVersion: .mark1_2, method: .post, path: path, parameters: parameters)
4043
let mapper = PostMapper()

Yosemite/YosemiteTests/Stores/SitePostStoreTests.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import XCTest
55

66
/// SitePostStore Unit Tests
77
///
8-
class SitePostStoreTests: XCTestCase {
8+
final class SitePostStoreTests: XCTestCase {
99

1010
/// Mockup Dispatcher!
1111
///

0 commit comments

Comments
 (0)