diff options
| author | Teddy Wing | 2017-06-14 15:37:35 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-06-14 15:37:35 +0200 |
| commit | f327ae88ebc45d291dc8485a0ade20a7e335740f (patch) | |
| tree | 8192c19285699b14e12e10b5fcd34fea8ebc550f /script | |
| parent | cd510d491181eb7a65559b9bbab600316e4f7b7b (diff) | |
| download | chouette-core-f327ae88ebc45d291dc8485a0ade20a7e335740f.tar.bz2 | |
TableBuilder: Trial refactor of `#build_links`
Instead of the existing code that we have for `if policy...` etc. in
`#build_links`, create a few new methods that ostensibly do the same
thing using my `Link` class.
It should create a bunch of `Link`s that can be combined with those that
come from the `[TABLE-OBJECT].action_links` (like those in
`ReferentialDecorator#action_links`). The combination of these will then
be turned into `link_to`s by our existing code.
Renamed the `actions` argument to `#build_links` to `links` because they
come into `table_builder_2` as a `links` variable and I got confused by
the name not being the same.
The policy checking is now in its own method,
`#actions_after_policy_check`. Need to verify that the logic is right,
but otherwise it keeps things separate. Might also want to write special
methods for each of the conditions between the `||`s.
TODO: Move the new private methods to a class. Also test them.
Refs #3479
Diffstat (limited to 'script')
0 files changed, 0 insertions, 0 deletions
