{"action":"created","issue":{"url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/issues/1238","repository_url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL","labels_url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/issues/1238/labels{/name}","comments_url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/issues/1238/comments","events_url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/issues/1238/events","html_url":"https://github.com/LoneGazebo/Community-Patch-DLL/issues/1238","id":132336789,"number":1238,"title":"City Player:GetCityByID(CityID city)","user":{"login":"GrayPockets","id":17136682,"avatar_url":"https://avatars.githubusercontent.com/u/17136682?v=3","gravatar_id":"","url":"https://api.github.com/users/GrayPockets","html_url":"https://github.com/GrayPockets","followers_url":"https://api.github.com/users/GrayPockets/followers","following_url":"https://api.github.com/users/GrayPockets/following{/other_user}","gists_url":"https://api.github.com/users/GrayPockets/gists{/gist_id}","starred_url":"https://api.github.com/users/GrayPockets/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/GrayPockets/subscriptions","organizations_url":"https://api.github.com/users/GrayPockets/orgs","repos_url":"https://api.github.com/users/GrayPockets/repos","events_url":"https://api.github.com/users/GrayPockets/events{/privacy}","received_events_url":"https://api.github.com/users/GrayPockets/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/labels/Fixed%20for%20upcoming%20version","name":"Fixed for upcoming version","color":"006b75"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":29,"created_at":"2016-02-09T05:44:39Z","updated_at":"2016-02-10T07:13:06Z","closed_at":"2016-02-10T05:04:23Z","body":"The [City Player:GetCityByID(CityID city)](http://modiki.civfanatics.com/index.php?title=Player.GetCityByID_(Civ5_API)) method does not appear to be working correctly when the CPP is installed. I specifically tested with the CPB but without EUI. I did not attempt with Core only.\r\n\r\nAs discussed in [this comments thread ](http://steamcommunity.com/sharedfiles/filedetails/?id=281728572), this method is always returning nil. It was causing a problem with the [Extra Victory Conditions](http://steamcommunity.com/sharedfiles/filedetails/?id=281728572) mod.\r\n\r\n**Steps to Reproduce**\r\n1. Install CPP /w CPB no EUI\r\n2. Install Extra Victory Conditions\r\n3. Create a Religion\r\n4. Review the Victory Progress overview\r\n5. See that it shows no followers listed for that religion.\r\n\r\n**Workaround** Use [iterator(City) Player:Cities()](http://modiki.civfanatics.com/index.php/Player.Cities_(Civ5_API))\r\n\r\nThis method was used by ReligiousVictory.lua. I am attaching two copies of ReligousVictory.lua, one working and one fixed.\r\n\r\n[ReligiousVictory.zip](https://github.com/LoneGazebo/Community-Patch-DLL/files/122641/ReligiousVictory.zip)"},"comment":{"url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/issues/comments/182233908","html_url":"https://github.com/LoneGazebo/Community-Patch-DLL/issues/1238#issuecomment-182233908","issue_url":"https://api.github.com/repos/LoneGazebo/Community-Patch-DLL/issues/1238","id":182233908,"user":{"login":"LoneGazebo","id":7849458,"avatar_url":"https://avatars.githubusercontent.com/u/7849458?v=3","gravatar_id":"","url":"https://api.github.com/users/LoneGazebo","html_url":"https://github.com/LoneGazebo","followers_url":"https://api.github.com/users/LoneGazebo/followers","following_url":"https://api.github.com/users/LoneGazebo/following{/other_user}","gists_url":"https://api.github.com/users/LoneGazebo/gists{/gist_id}","starred_url":"https://api.github.com/users/LoneGazebo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LoneGazebo/subscriptions","organizations_url":"https://api.github.com/users/LoneGazebo/orgs","repos_url":"https://api.github.com/users/LoneGazebo/repos","events_url":"https://api.github.com/users/LoneGazebo/events{/privacy}","received_events_url":"https://api.github.com/users/LoneGazebo/received_events","type":"User","site_admin":false},"created_at":"2016-02-10T07:13:06Z","updated_at":"2016-02-10T07:13:06Z","body":"@Iamblichos , that's correct.\r\n\r\n@ilteroi , looks better than mine, yep (I didn't know that <1000 mattered, tbh). Shouldn't be an immediate issue, but I'll add that in to the next version.\r\n\r\nFeel free to add to yours and I'll grab in the next merge.\r\nG"}}