{"issue":{"state":"open","created_at":"2011-11-29T09:42:10Z","updated_at":"2011-11-29T09:42:12Z","user":{"gravatar_id":"256406243346b9253f8610fb1d1e05f6","id":1215373,"url":"https://api.github.com/users/importer","login":"importer","avatar_url":"https://secure.gravatar.com/avatar/256406243346b9253f8610fb1d1e05f6?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"comments":2,"body":"My client wanted simply to add a Location field to capture where people are from, instead of their website (which he couldn't care about). I then realised that the Comments system is not extendible, because the Front-End part has not been written in a generic way. Perhaps we can modify the core to allow a 'feEditable' field, so the Comments.php will scan for that and construct the widgets from using the BE info, instead of the FE hard-coded manual values. This also means that we can target additional routines to specific field types, rather than specific fields, e.g. the routing to add http:// to the website field, rather make it the url type fields. Then I noticed that the language strings for the comment fields are taken from the front-end, when instead these can then be taken from the BE definitions (no need for double language names). This would be similar to the membership fields.\n\nIt also affects the listComments method, as it renders in a pre-defined way, so that should be extendible too (move website and other fields into the list comments hook).\n\nRelated issues: #2332\n\n--- _Originally created by thyon on August 20th, 2010, at 10:35am (ID 2451)_","assignee":null,"id":2382058,"labels":[{"name":"Accepted","color":"6cc644","url":"https://api.github.com/repos/contao/test/labels/Accepted"},{"name":"comments","color":"e6e6e6","url":"https://api.github.com/repos/contao/test/labels/comments"},{"name":"Feature","color":"4183c4","url":"https://api.github.com/repos/contao/test/labels/Feature"}],"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"number":2451,"title":"Comments System: allow custom fields","closed_at":null,"html_url":"https://github.com/contao/test/issues/2451","url":"https://api.github.com/repos/contao/test/issues/2451","milestone":null},"action":"created","comment":{"created_at":"2011-11-29T09:42:12Z","updated_at":"2011-11-29T09:42:12Z","user":{"gravatar_id":"256406243346b9253f8610fb1d1e05f6","id":1215373,"url":"https://api.github.com/users/importer","login":"importer","avatar_url":"https://secure.gravatar.com/avatar/256406243346b9253f8610fb1d1e05f6?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"body":"Kewl Thanks. \n\nPS. I can't understand German, so I usually skip all the tickets that are not in English.\n\n--- _Originally created by thyon on August 20th, 2010, at 03:55pm_","id":2925527,"url":"https://api.github.com/repos/contao/test/issues/comments/2925527"}}