{"action":"created","issue":{"url":"https://api.github.com/repos/HTBox/allReady/issues/1415","repository_url":"https://api.github.com/repos/HTBox/allReady","labels_url":"https://api.github.com/repos/HTBox/allReady/issues/1415/labels{/name}","comments_url":"https://api.github.com/repos/HTBox/allReady/issues/1415/comments","events_url":"https://api.github.com/repos/HTBox/allReady/issues/1415/events","html_url":"https://github.com/HTBox/allReady/pull/1415","id":186157857,"number":1415,"title":"#1346 Fix copy contact info","user":{"login":"nemesv","id":251330,"avatar_url":"https://avatars.githubusercontent.com/u/251330?v=3","gravatar_id":"","url":"https://api.github.com/users/nemesv","html_url":"https://github.com/nemesv","followers_url":"https://api.github.com/users/nemesv/followers","following_url":"https://api.github.com/users/nemesv/following{/other_user}","gists_url":"https://api.github.com/users/nemesv/gists{/gist_id}","starred_url":"https://api.github.com/users/nemesv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nemesv/subscriptions","organizations_url":"https://api.github.com/users/nemesv/orgs","repos_url":"https://api.github.com/users/nemesv/repos","events_url":"https://api.github.com/users/nemesv/events{/privacy}","received_events_url":"https://api.github.com/users/nemesv/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-10-30T19:43:48Z","updated_at":"2016-10-31T16:01:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/HTBox/allReady/pulls/1415","html_url":"https://github.com/HTBox/allReady/pull/1415","diff_url":"https://github.com/HTBox/allReady/pull/1415.diff","patch_url":"https://github.com/HTBox/allReady/pull/1415.patch"},"body":"The commit ad2bb418 changed the `PostalCode` from `PostalCodeGeo` to `string`.  As a part of this refactoring all the `.ThenInclude(pc => pc.PostalCode)` removed, but not in the `OrganizationContactQueryHandler`.\n\nSo `PostalCode`  is no longer a navigation property on the `Location` object\nand currently calling `.ThenInclude(pc => pc.PostalCode)` results in an \n\n```\nSystem.ArgumentNullException: Value cannot be null.\nParameter name: source\n   at System.Linq.Enumerable.ToArray[TSource](IEnumerable`1 source)\n   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.BindPropertyExpressionCore[TResult](Expression propertyExpression, IQuerySource querySource, Func`3 propertyBinder)\n   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.BindNavigationPathPropertyExpression[TResult](Expression propertyExpression, Func`3 propertyBinder)\n   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.<IncludeNavigations>b__42_0(IncludeResultOperator includeResultOperator)\n   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()\n   at System.Linq.Buffer`1..ctor(IEnumerable`1 source)\n   at System.Linq.OrderedEnumerable`1.<GetEnumerator>d__1.MoveNext()\n   at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()\n   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)\n   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)\n   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.IncludeNavigations(QueryModel queryModel)\n   at Microsoft.EntityFrameworkCore.Query.EntityQueryModelVisitor.CreateAsyncQueryExecutor[TResult](QueryModel queryModel)\n   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddAsyncQuery[TResult](Object cacheKey, Func`1 compiler)\n   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.ExecuteAsync[TResult](Expression query, CancellationToken cancellationToken)\n```\n\nRemoving the unnecessary `ThenInclude` fixes this exception and the #1346 \"Copy Contact\" feature.\n"},"comment":{"url":"https://api.github.com/repos/HTBox/allReady/issues/comments/257335407","html_url":"https://github.com/HTBox/allReady/pull/1415#issuecomment-257335407","issue_url":"https://api.github.com/repos/HTBox/allReady/issues/1415","id":257335407,"user":{"login":"tonysurma","id":5067595,"avatar_url":"https://avatars.githubusercontent.com/u/5067595?v=3","gravatar_id":"","url":"https://api.github.com/users/tonysurma","html_url":"https://github.com/tonysurma","followers_url":"https://api.github.com/users/tonysurma/followers","following_url":"https://api.github.com/users/tonysurma/following{/other_user}","gists_url":"https://api.github.com/users/tonysurma/gists{/gist_id}","starred_url":"https://api.github.com/users/tonysurma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tonysurma/subscriptions","organizations_url":"https://api.github.com/users/tonysurma/orgs","repos_url":"https://api.github.com/users/tonysurma/repos","events_url":"https://api.github.com/users/tonysurma/events{/privacy}","received_events_url":"https://api.github.com/users/tonysurma/received_events","type":"User","site_admin":false},"created_at":"2016-10-31T16:01:20Z","updated_at":"2016-10-31T16:01:20Z","body":"Thanks @nemesv again for your contribution!  Also thank you for the explanation so others can understand and learn"}}
{"actor":{"display_login":"tonysurma"}}