Github user mitchell852 commented on a diff in the pull request:
https://github.com/apache/incubator-trafficcontrol/pull/141#discussion_r93509215
--- Diff: traffic_ops/app/lib/UI/ConfigFiles.pm ---
@@ -312,6 +312,9 @@ sub param_data {
if ( $row->parameter->name eq "location" ) {
--- End diff --
tbh, when I see this kind of stuff I start to think we've overloaded the
parameter table. parameters with config_file != null should imo be parameters
that are written to that config file but for these 2 special cases (location
and scope) we key off name to skip over them.
it almost feels like a configfiles table could be justified with name,
location, scope....but we can hold that thought for now...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
