Re: [Kimchi-devel] [PATCH 3/3] issue #461: Add 'metalink' support for YUM repositories
On 26-03-2015 16:55, Aline Manera wrote:
We can use '' (empty string) as the default value while
getting the
information from the data sent and simplify that code in a similar way
we did on addRepo()
When adding a new repository, it doesn't matter if the user didn't
specify one of those fields or if they specified an empty string; it's
all "empty", and we always convert the empty value to an empty string
when adding a new repo.
However, when editing an existing repository, which is the case you
mentioned here, there _is_ a difference between not specifying a value
(i.e. the user doesn't want to change it) and specifying an empty string
(i.e. the user wants to clear that field). That's why we need to handle
a value which wasn't specified as None, and not as an empty string.
Actually, I just followed along the existing code regarding base URL and
mirror list, which uses None as the default (i.e. non-incoming) value,
but I guess that is the real reason it was implemented like that.