{"comment":{"url":"https://api.github.com/repos/ninab/Personas/comments/9991298","html_url":"https://github.com/ninab/Personas/commit/5273130dd83f193543ed07fd0ba8ffeaba5583e1#commitcomment-9991298","id":9991298,"user":{"login":"dewolfe001","id":4086452,"avatar_url":"https://avatars.githubusercontent.com/u/4086452?v=3","gravatar_id":"","url":"https://api.github.com/users/dewolfe001","html_url":"https://github.com/dewolfe001","followers_url":"https://api.github.com/users/dewolfe001/followers","following_url":"https://api.github.com/users/dewolfe001/following{/other_user}","gists_url":"https://api.github.com/users/dewolfe001/gists{/gist_id}","starred_url":"https://api.github.com/users/dewolfe001/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dewolfe001/subscriptions","organizations_url":"https://api.github.com/users/dewolfe001/orgs","repos_url":"https://api.github.com/users/dewolfe001/repos","events_url":"https://api.github.com/users/dewolfe001/events{/privacy}","received_events_url":"https://api.github.com/users/dewolfe001/received_events","type":"User","site_admin":false},"position":70,"line":93,"path":"PKP/plugins/PDFJSPlugin/PDFJSPlugin.inc.php","commit_id":"5273130dd83f193543ed07fd0ba8ffeaba5583e1","created_at":"2015-03-02T18:39:00Z","updated_at":"2015-03-02T18:39:00Z","body":"That is an expensive way to get the data is present in the requested object. Why should there be additional processing called upon when the object in the iteration of the foreach loop already contains what is required?\r\n\r\neg. this is what is in the ArticleFile Object. When will there be an occasion that this information is inaccurate and the getter is more efficient and more accurate than was is present inside of the object? \r\nArticleFile Object\r\n(\r\n    [_data] => Array\r\n        (\r\n            [fileId] => 6\r\n            [sourceFileId] => 4\r\n            [revision] => 1\r\n            [submissionId] => 2\r\n            [fileName] => 2-6-1-RV.pdf\r\n            [filetype] => application/pdf\r\n            [fileSize] => 1647575\r\n            [originalFileName] => 2-4-1-SM.pdf\r\n            [fileStage] => 2\r\n            [dateUploaded] => 2015-01-20 14:53:08\r\n            [dateModified] => 2015-01-20 14:53:08\r\n            [round] => 1\r\n        )\r\n\r\n    [_hasLoadableAdapters] => \r\n    [_metadataExtractionAdapters] => Array\r\n        (\r\n        )\r\n\r\n    [_extractionAdaptersLoaded] => \r\n    [_metadataInjectionAdapters] => Array\r\n        (\r\n        )\r\n\r\n    [_injectionAdaptersLoaded] => \r\n)"}}