(11:40:05 AM) jshaughn: ips, did we ever complete broad coverage for the dbl-click/name link approach? (11:40:52 AM) ips: jshaughn: i don't think so. we have double-click everywhere that uses TableSection (11:41:29 AM) ips: but we probably need to go through and add the name links to all the pages that use TableSection (11:42:05 AM) jshaughn: ok (11:42:06 AM) ips: may be worth adding an abstract method in TableSection to force subclasses to provide a link (11:43:12 AM) ips: protected getDetailsLink(int id) (11:43:45 AM) ips: actually the link will always be tableSectionsViewPath + "/" + id right? (11:44:15 AM) jshaughn: ips, I think so (11:44:32 AM) ips: so maybe all that's needed would be protected abstract getLinkColumnFieldName() (11:44:44 AM) ips: which could have the default impl: return "name"; (11:45:07 AM) jshaughn: ips, yeah, I think a base impl would cover most, if not all cases
Done - [master df6684a].
Bookkeeping - closing bug - fixed in recent release.