Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Commit 211471d

Browse files
Merge pull request #721 from github-for-unity/more-serialization-magic
Add more options to SimpleJson
2 parents d823376 + 9de0b16 commit 211471d

File tree

14 files changed

+236
-43
lines changed

14 files changed

+236
-43
lines changed

script

src/.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -311,4 +311,5 @@ sysinfo.txt
311311

312312
# Builds
313313
*.apk
314-
*.unitypackage
314+
*.unitypackage
315+
UnityExtension/**/manifest.json

src/GitHub.Api/Authentication/Keychain.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ private void LoadConnectionsFromDisk()
198198
var json = cachePath.ReadAllText();
199199
try
200200
{
201-
var conns = SimpleJson.DeserializeObject<Connection[]>(json);
201+
var conns = json.FromJson<Connection[]>();
202202
UpdateConnections(conns);
203203
}
204204
catch (IOException ex)
@@ -219,7 +219,7 @@ private void SaveConnectionsToDisk(bool raiseChangedEvent = true)
219219
//logger.Trace("WriteCacheToDisk Count:{0} Path:{1}", connectionCache.Count, cachePath.ToString());
220220
try
221221
{
222-
var json = SimpleJson.SerializeObject(connections.Values.ToArray());
222+
var json = connections.Values.ToJson();
223223
cachePath.WriteAllText(json);
224224
}
225225
catch (IOException ex)

src/GitHub.Api/Cache/CacheContainer.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ public DateTimeOffset UpdatedTime
164164
if (!updatedTimeValue.HasValue)
165165
{
166166
DateTimeOffset result;
167-
if (DateTimeOffset.TryParseExact(updatedTimeString, Constants.Iso8601Format, CultureInfo.InvariantCulture, DateTimeStyles.None, out result))
167+
if (DateTimeOffset.TryParseExact(updatedTimeString, Constants.Iso8601Formats, CultureInfo.InvariantCulture, DateTimeStyles.None, out result))
168168
{
169169
updatedTimeValue = result;
170170
}

src/GitHub.Api/Git/GitLogEntry.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ public DateTimeOffset Time
8282
if (!timeValue.HasValue)
8383
{
8484
DateTimeOffset result;
85-
if (DateTimeOffset.TryParseExact(TimeString, Constants.Iso8601Format, CultureInfo.InvariantCulture,DateTimeStyles.None, out result))
85+
if (DateTimeOffset.TryParseExact(TimeString, Constants.Iso8601Formats, CultureInfo.InvariantCulture,DateTimeStyles.None, out result))
8686
{
8787
timeValue = result;
8888
}
@@ -109,7 +109,7 @@ public DateTimeOffset CommitTime
109109
if (!commitTimeValue.HasValue)
110110
{
111111
DateTimeOffset result;
112-
if (DateTimeOffset.TryParseExact(CommitTimeString, Constants.Iso8601Format, CultureInfo.InvariantCulture, DateTimeStyles.None, out result))
112+
if (DateTimeOffset.TryParseExact(CommitTimeString, Constants.Iso8601Formats, CultureInfo.InvariantCulture, DateTimeStyles.None, out result))
113113
{
114114
commitTimeValue = result;
115115
}

src/GitHub.Api/Helpers/Constants.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,12 @@ static class Constants
1010
public const string GitInstallPathKey = "GitInstallPath";
1111
public const string TraceLoggingKey = "EnableTraceLogging";
1212
public const string WebTimeoutKey = "WebTimeout";
13-
public const string Iso8601Format = "yyyy-MM-ddTHH\\:mm\\:ss.fffffffzzz";
13+
public const string Iso8601Format = @"yyyy-MM-dd\THH\:mm\:ss.fffzzz";
14+
public static readonly string[] Iso8601Formats = {
15+
@"yyyy-MM-dd\THH\:mm\:ss.fffffffzzz",
16+
@"yyyy-MM-dd\THH\:mm\:ss.fffzzz",
17+
@"yyyy-MM-dd\THH\:mm\:sszzz"
18+
};
1419

1520
public static readonly Version MinimumGitVersion = new Version(2, 11, 0);
1621
public static readonly Version MinimumGitLfsVersion = new Version(2, 3, 4);

0 commit comments

Comments
 (0)