Opened 5 years ago

Last modified 4 years ago

#3779 new defect

quadruplicated definition of "do_doctest_gettext_workaround()"

Reported by: pmav99 Owned by: grass-dev@…
Priority: normal Milestone: 7.8.3
Component: Tests Version: svn-trunk
Keywords: Cc:
CPU: Unspecified Platform: Unspecified

Description

do_doctest_gettext_workaround() is being defined 4 times in the codebase.

AFAI can tell, apart from the docstrings there are no differences whatsoever.

$  ag 'def do_doctest'
gui/wxpython/tools/build_modules_xml.py
33:def do_doctest_gettext_workaround():

gui/wxpython/core/utils.py
1155:def do_doctest_gettext_workaround():

gui/wxpython/core/toolboxes.py
774:def do_doctest_gettext_workaround():

lib/python/gunittest/utils.py
35:def do_doctest_gettext_workaround():

lib/python/gunittest/utils.py seems like the most reasonable place to keep this definition. The other definitions should be removed and the imports in the gui library should be updated. The docstring at gui/wxpython/core/toolboxes.py is the most informative one.

Change History (2)

comment:1 by neteler, 5 years ago

If possible please provide a patch, thanks

comment:2 by neteler, 4 years ago

Milestone: 7.8.3
Note: See TracTickets for help on using tickets.