{"action":"closed","issue":{"url":"https://api.github.com/repos/georgringer/news/issues/193","repository_url":"https://api.github.com/repos/georgringer/news","labels_url":"https://api.github.com/repos/georgringer/news/issues/193/labels{/name}","comments_url":"https://api.github.com/repos/georgringer/news/issues/193/comments","events_url":"https://api.github.com/repos/georgringer/news/issues/193/events","html_url":"https://github.com/georgringer/news/issues/193","id":197239636,"number":193,"title":"No TemplateLayouts in Templateselector in news 5.3.1","user":{"login":"Schwuuuuup","id":16288510,"avatar_url":"https://avatars.githubusercontent.com/u/16288510?v=3","gravatar_id":"","url":"https://api.github.com/users/Schwuuuuup","html_url":"https://github.com/Schwuuuuup","followers_url":"https://api.github.com/users/Schwuuuuup/followers","following_url":"https://api.github.com/users/Schwuuuuup/following{/other_user}","gists_url":"https://api.github.com/users/Schwuuuuup/gists{/gist_id}","starred_url":"https://api.github.com/users/Schwuuuuup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Schwuuuuup/subscriptions","organizations_url":"https://api.github.com/users/Schwuuuuup/orgs","repos_url":"https://api.github.com/users/Schwuuuuup/repos","events_url":"https://api.github.com/users/Schwuuuuup/events{/privacy}","received_events_url":"https://api.github.com/users/Schwuuuuup/received_events","type":"User","site_admin":false},"labels":[{"id":49767491,"url":"https://api.github.com/repos/georgringer/news/labels/Bug","name":"Bug","color":"fc2929","default":false},{"id":449719529,"url":"https://api.github.com/repos/georgringer/news/labels/Confirmed","name":"Confirmed","color":"e99695","default":false},{"id":49767495,"url":"https://api.github.com/repos/georgringer/news/labels/Needs%20feedback%20from%20PR%20author","name":"Needs feedback from PR author","color":"cc317c","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-22T18:38:11Z","updated_at":"2016-12-29T11:39:44Z","closed_at":"2016-12-29T11:39:44Z","body":"Hi,\r\n\r\nI'm aware of Issue TYPO3 Forge Issue 78019, but unfortunately I need compatibility6 because of simulatestatic (Having 50000+ Pages and 3000+ Users doesn't allow me to change the URL scheme out of nowhere).\r\n\r\nWould it be possible to include compatibility to compatibility6?\r\n\r\nAlso, I'm not sure if t's only this: I patched the version like This:\r\n\r\nin function user_templateLayout(array &$config) I changed\r\n\r\nthis:\r\n\r\n` $pageId = $this->getPageId($config['flexParentDatabaseRow']['pid']);`\r\n\r\nto:\r\n` $pageId = $this->getPageId($config['row']['uid']);`\r\n\r\nBut this returns the ID of the tt_content record. \r\nI think there is a bug in getPageId.\r\nI don't really understand the comment\r\n\r\n`* Get page id, if negative, then it is a \"after record\"`\r\nThe condition does not check for \"negaive\" but for \"greater then zero\", so when I swap the > for an <, it works:\r\n```\r\n protected function getPageId($pid)\r\n    {\r\n        $pid = (int)$pid;\r\n\r\n        if ($pid < 0) {\r\n            return $pid;\r\n        } else {\r\n            $row = BackendUtilityCore::getRecord('tt_content', abs($pid), 'uid,pid');\r\n            return $row['pid'];\r\n        }\r\n    }\r\n```"}}
{"actor":{"display_login":"georgringer"}}