New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Vfolders namespace #5221
Vfolders namespace #5221
Conversation
04ce8c1
to
9186ef0
Compare
# or later license. See the LICENSE file for a copy of the license and the | ||
# AUTHORS file for copyright and authorship information. | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: only one blank line before imports.
edc79fa
to
f3e344f
Compare
def test_views_vf_translate(vfolder_views, settings): | ||
from tp import _test_browse_view, _test_translate_view | ||
test_type, tp, request, response, kwargs = vfolder_views | ||
if test_type == "browse": |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is not necessary
449c003
to
86388e5
Compare
# A clean pootle path is a pootle path without any virtual folder name on | ||
# it. For example /af/test_vfolders/browser/chrome/ is the corresponding | ||
# clean pootle path for /af/test_vfolders/browser/vfolder8/chrome/ | ||
vfolder, clean_pootle_path = extract_vfolder_from_path(pootle_path) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given that this is dropped you can probably further simplify the surrounding code a bit more now that clean_pootle_path
and pootle_path
are the same.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is addressed.
21a01cd
to
fea850a
Compare
if filename: | ||
pootle_path = pootle_path + filename | ||
clean_pootle_path = clean_pootle_path + filename | ||
pootle_path = pootle_path + filename |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Repeated code
fa2e6cb
to
02bc9b0
Compare
02bc9b0
to
e142ed1
Compare
lgtm |
massive thanks for all your hard working reviewing and explaining our bugs back to us @unho ! |
addresses concerns raised in #5217 (Namespacing problems with virtualfolders)