table-plus fails to name "count" columns correctly and uniquely
Reported by mkosma | June 9th, 2009 @ 07:44 AM | in Beyond Hobo 1.0
To replicate, create a model with links to two other tables. In my case, projects have custodians and file_sets.
<index-page>
<collection: replace>
<table-plus fields="this, custodians.count, file_sets.count">
<custodians_count-heading-link:>Custodians</custodians_count-heading-link:>
<custodians_count-view: class="numeric-field"><%= number-with-delimiter(this) %></custodians_count-view:>
<file_sets_count-heading-link:>File Sets</file_sets_count-heading-link:>
<file_sets_count-view: class="numeric-field"><%= number_with_delimiter(this) %></file_sets_count-view:>
</table-plus>
</collection:>
</index-page>
The -heading tags are generated properly and the above substitution works. However, Rapid incorrectly truncates the class for
table entries to "count-view" instead of, e.g.,
"file_sets_count-view", as illustrated in the html below (and
attached in a more complete version). As a result, the above
attempt to substitute for file_sets_count-view and
custodians_count-view has no effect.
|
Comments and changes to this ticket
-
Bryan Larsen June 16th, 2009 @ 11:19 PM
Notes to self as I break for the day:
table uses
#{this_field}.view
. Source is ultimately get_field_path in hobo.rb. -
Tom Locke October 24th, 2009 @ 06:08 PM
- State changed from new to open
- Milestone set to Beyond Hobo 1.0
In fact these 'count' columns were never actually designed in, but rather were a happy accident of the implementation. I guess they are pretty useful though :)
-
Bryan Larsen December 29th, 2009 @ 06:30 PM
- State changed from open to resolved
(from [1d8b5a9c1a2bd3e9442726a5482659d7eec223c7]) [#592 state:resolved] [#436 state:resolved] name "foos.count" view in a table or field-list "foos-count-view" instead of "count-view" http://github.com/tablatom/hobo/commit/1d8b5a9c1a2bd3e9442726a54826...
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
People watching this ticket
Attachments
Tags
Referenced by
- 592 <table> cuts fields with dots This is probably a duplicate of #436, and the fix isn't e...
- 592 <table> cuts fields with dots BTW, does this also lead to a fix for #436? Probably nice...
- 592 <table> cuts fields with dots This also fixes #436.
- 592 <table> cuts fields with dots (from [1d8b5a9c1a2bd3e9442726a5482659d7eec223c7]) [#592 s...
- 436 table-plus fails to name "count" columns correctly and uniquely (from [1d8b5a9c1a2bd3e9442726a5482659d7eec223c7]) [#592 s...