<feed xmlns='http://www.w3.org/2005/Atom'>
<title>chouette-core/app/helpers, branch check_policy_specs</title>
<subtitle>Chouette manage transport static data</subtitle>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/'/>
<entry>
<title>Fix sort Line transport submode with special char</title>
<updated>2017-06-23T10:06:02+00:00</updated>
<author>
<name>Xinhui</name>
</author>
<published>2017-06-23T10:06:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=763ba5430c9babc38ef816f9cd3b4abe9b250f88'/>
<id>763ba5430c9babc38ef816f9cd3b4abe9b250f88</id>
<content type='text'>
Refs #3851
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Refs #3851
</pre>
</div>
</content>
</entry>
<entry>
<title>TableBuilderHelper: Get rid of extra unnecessary whitespace</title>
<updated>2017-06-19T16:02:20+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-19T16:02:20+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=443f281c763670aeee62109cf7bd510f1d88d7c6'/>
<id>443f281c763670aeee62109cf7bd510f1d88d7c6</id>
<content type='text'>
Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>Lines#show: Render header links from #action_links</title>
<updated>2017-06-19T15:42:05+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-19T15:24:15+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=d4de52d63f400f30dabb7034e14202f5c9bd0192'/>
<id>d4de52d63f400f30dabb7034e14202f5c9bd0192</id>
<content type='text'>
Get the header links from the decorator so we can reuse them instead of
defining them directly in the template.

Add an option to the `destroy_link_content` Rails helper method to allow
us to specify alternate translation keys, giving us different text in
the button label. Added this because the template in question uses a
different label than the others (usually it's `actions.destroy`).

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Get the header links from the decorator so we can reuse them instead of
defining them directly in the template.

Add an option to the `destroy_link_content` Rails helper method to allow
us to specify alternate translation keys, giving us different text in
the button label. Added this because the template in question uses a
different label than the others (usually it's `actions.destroy`).

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>TableBuilder: Add documentation</title>
<updated>2017-06-19T09:00:45+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-19T09:00:45+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=88f1d1c6c4dd82404c124ec15ab56513289c2210'/>
<id>88f1d1c6c4dd82404c124ec15ab56513289c2210</id>
<content type='text'>
Add a few doc comments to the table builder to provide a little more
detailed explanation of the code.

TableBuilderHelper::URL:
* Describe dependency on `current_referential`

TableBuilderHelper:
* Add doc comments to the `collection` and `columns` arguments
* Add a description of the module
* Detail external global variable dependencies
* Provide an example, stolen from one of the tests

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add a few doc comments to the table builder to provide a little more
detailed explanation of the code.

TableBuilderHelper::URL:
* Describe dependency on `current_referential`

TableBuilderHelper:
* Add doc comments to the `collection` and `columns` arguments
* Add a description of the module
* Detail external global variable dependencies
* Provide an example, stolen from one of the tests

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>TableBuilderHelper::Column: Use `@name` consistently</title>
<updated>2017-06-19T08:47:07+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-19T08:47:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=7887e9f44eef32382fce885d9e3270e390444bad'/>
<id>7887e9f44eef32382fce885d9e3270e390444bad</id>
<content type='text'>
We were using both the instance variable and the accessor on the same
line. Pick one and be consistent.

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
We were using both the instance variable and the accessor on the same
line. Pick one and be consistent.

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>MultipleSelectionToolbox: Clean up code; Add to workbenches/show.html</title>
<updated>2017-06-15T17:07:09+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-15T17:07:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=6e17029c934abf7e09b05245e7bb25c3c6480fe9'/>
<id>6e17029c934abf7e09b05245e7bb25c3c6480fe9</id>
<content type='text'>
Keep the helper code globally the same.

Changes:
* Remove `if action == :edit` since we never execute that branch. In
  fact, the only place we ever use the multiple selection toolbox in
  conjunction with the table builder is on Workbenches#show, and that
  page only uses a :delete action.
* Rename `dPath` to be clearer
* Add some whitespace to improve readability
* Rename `actitem` to `action_link` for better readability
* Connect the links and label with `#+` instead of `#concat`

Add the toolbox to "workbenches/show.html.slim" to get the page looking
like it was before I changed the table builder.

Needed to wrap the table and toolbox in a `&lt;div class="select_table"&gt;`
in order for the JavaScript part of it to work (otherwise selection
wouldn't activate the link, and likely other stuff would be messed up
too).

Now that the table builder and the selection toolbox are separate, it
makes sense to put the "select_table" `&lt;div&gt;` in the template.

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Keep the helper code globally the same.

Changes:
* Remove `if action == :edit` since we never execute that branch. In
  fact, the only place we ever use the multiple selection toolbox in
  conjunction with the table builder is on Workbenches#show, and that
  page only uses a :delete action.
* Rename `dPath` to be clearer
* Add some whitespace to improve readability
* Rename `actitem` to `action_link` for better readability
* Connect the links and label with `#+` instead of `#concat`

Add the toolbox to "workbenches/show.html.slim" to get the page looking
like it was before I changed the table builder.

Needed to wrap the table and toolbox in a `&lt;div class="select_table"&gt;`
in order for the JavaScript part of it to work (otherwise selection
wouldn't activate the link, and likely other stuff would be messed up
too).

Now that the table builder and the selection toolbox are separate, it
makes sense to put the "select_table" `&lt;div&gt;` in the template.

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>Add `MultipleSelectionToolboxHelper`</title>
<updated>2017-06-15T17:03:27+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-15T16:45:57+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=5a54186ab087484334d6ff15976d7198ee79371a'/>
<id>5a54186ab087484334d6ff15976d7198ee79371a</id>
<content type='text'>
Port the `select_toolbox` from `NewapplicationHelper`. For now the code
is the same. Just the name and method comment have changed.

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Port the `select_toolbox` from `NewapplicationHelper`. For now the code
is the same. Just the name and method comment have changed.

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>ReferentialDecorator: Handle non-`Link` buttons</title>
<updated>2017-06-15T15:44:55+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-15T15:44:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=1d489cef3b9aeb474071b89be6d256e554182863'/>
<id>1d489cef3b9aeb474071b89be6d256e554182863</id>
<content type='text'>
Referentials#show has a button "Purger" that isn't a link. Instead, it's
a `&lt;button&gt;` element that opens a modal.

Previously, we only knew how to put `Link`s in the `#action_links`, and
only these could then be rendered in the header bar as buttons.

This change allows us to accept non-Link objects. Since the button is
rather simple, I decided to create a simple `HTMLElement` class to
represent it and output it in the resulting HTML. The class uses the
`#content_tag` `ActionView` helper to render the final HTML.

Update "referentials/show.html.slim" to output the button correctly in
addition to the normal links.

Update `TableBuilderHelper#build_links` to not add anything that's not a
`Link` to the gear menu. This allows us to display the "Purger" button
on the Referentials#show page but not in the table on Workbenches#show.

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Referentials#show has a button "Purger" that isn't a link. Instead, it's
a `&lt;button&gt;` element that opens a modal.

Previously, we only knew how to put `Link`s in the `#action_links`, and
only these could then be rendered in the header bar as buttons.

This change allows us to accept non-Link objects. Since the button is
rather simple, I decided to create a simple `HTMLElement` class to
represent it and output it in the resulting HTML. The class uses the
`#content_tag` `ActionView` helper to render the final HTML.

Update "referentials/show.html.slim" to output the button correctly in
addition to the normal links.

Update `TableBuilderHelper#build_links` to not add anything that's not a
`Link` to the gear menu. This allows us to display the "Purger" button
on the Referentials#show page but not in the table on Workbenches#show.

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>NewapplicationHelper: Remove my investigation comments</title>
<updated>2017-06-15T14:50:37+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-15T14:50:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=97c8cd58c1d3fd684f41135ac1774fac4cc46ea1'/>
<id>97c8cd58c1d3fd684f41135ac1774fac4cc46ea1</id>
<content type='text'>
I added these comments when I was reading the code and trying to
understand it. Now I basically understand all of it, and we've ported
the code to `TableBuilderHelper`. These comments can be removed to
revert the file back to how it looked before I messed with it.

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
I added these comments when I was reading the code and trying to
understand it. Now I basically understand all of it, and we've ported
the code to `TableBuilderHelper`. These comments can be removed to
revert the file back to how it looked before I messed with it.

Refs #3479
</pre>
</div>
</content>
</entry>
<entry>
<title>TableBuilderHelper#table_builder_2: Clean up arguments &amp; add docs</title>
<updated>2017-06-15T14:42:42+00:00</updated>
<author>
<name>Teddy Wing</name>
</author>
<published>2017-06-15T14:42:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.teddywing.com/fork/chouette-core/commit/?id=73f8e3bb8a5e6ac47874ce75ed3bf81c63a633dd'/>
<id>73f8e3bb8a5e6ac47874ce75ed3bf81c63a633dd</id>
<content type='text'>
Remove unnecessary arguments and document the ones we are using.

`current_referential` has been removed. Ultimately decided that it was
simpler to just rely on the existing method that's magically "imported"
instead of passing it directly. If you vote to pass it in as an arg,
though, I'm all for it.

Refs #3479
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remove unnecessary arguments and document the ones we are using.

`current_referential` has been removed. Ultimately decided that it was
simpler to just rely on the existing method that's magically "imported"
instead of passing it directly. If you vote to pass it in as an arg,
though, I'm all for it.

Refs #3479
</pre>
</div>
</content>
</entry>
</feed>
