Since updating the filtering options for pkgs on ppas to use more
zope.formlib stuff, we can no-longer explicitly declare the null-option,
but need to rely on zope's built-in null option for choices (which gets
added irrespectively for non-required choices).
== Proposed fix ==
Remove our explicit 'any' option for status and series filters and
instead rely on zope's built-in None option.
== Pre-implementation notes ==
Discussed briefly with Celso to see how we had handled this in other
situations.
== Implementation details ==
I ended up with two near identical properties - selected_status_filter
and selected_series_filter - hence drying up with the
getSelectedFilterValue() method.
== Tests ==
bin/test -vv -t archive-views -t stories/ppa
== Demo and Q/A ==
There should no longer be a '(no value)' option at:
= Summary =
This branch is a fix for bug 429263.
Since updating the filtering options for pkgs on ppas to use more
zope.formlib stuff, we can no-longer explicitly declare the null-option,
but need to rely on zope's built-in null option for choices (which gets
added irrespectively for non-required choices).
== Proposed fix ==
Remove our explicit 'any' option for status and series filters and
instead rely on zope's built-in None option.
== Pre-implementation notes ==
Discussed briefly with Celso to see how we had handled this in other
situations.
== Implementation details ==
I ended up with two near identical properties - selected_ status_ filter series_ filter - hence drying up with the erValue( ) method.
and selected_
getSelectedFilt
== Tests ==
bin/test -vv -t archive-views -t stories/ppa
== Demo and Q/A ==
There should no longer be a '(no value)' option at:
Local demo, play with filtering at: /launchpad. dev/~cprov/ +archive/ ppa /launchpad. dev/~cprov/ +archive/ ppa/+packages /launchpad. dev/~cprov/ +archive/ ppa/+copy- packages /launchpad. dev/~cprov/ +archive/ ppa/+delete- packages
https:/
https:/
https:/
https:/
On edge: similar sub-urls from:
https:/ /edge.launchpad .net/~cprov/ +archive/ ppa
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: soyuz/stories/ ppa/xx- ubuntu- ppas.txt soyuz/browser/ tests/archive- views.txt soyuz/stories/ ppa/xx- delete- packages. txt soyuz/stories/ ppa/xx- copy-packages. txt soyuz/stories/ ppa/xx- ppa-packages. txt soyuz/browser/ archive. py
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
--
Michael