name 'QtGui' is not defined - but it is!

[mlampert]

Actually I don't see any need to implement a translate() function. It would suffice to directly use QCoreApplication.translate where the third and fourth parameter is omitted to be source compatible with Qt5.

So you're saying the try:except: block is redundant?
And it is OK to import QtGui even if FreeCAD is run without an UI?

[wmayer]

So you're saying the try:except: block is redundant?

The whole implementation is redundant.

And it is OK to import QtGui even if FreeCAD is run without an UI?

That's not a problem because the translation facility is part of the QtCore module and not QtGui. For any reason most of the coders use QtGui.QApplication but the translate() function is part of the base class in QtCore.QCoreApplication.

[mlampert]

So you're saying the try:except: block is redundant?

The whole implementation is redundant.

And it is OK to import QtGui even if FreeCAD is run without an UI?

That's not a problem because the translation facility is part of the QtCore module and not QtGui. For any reason most of the coders use QtGui.QApplication but the translate() function is part of the base class in QtCore.QCoreApplication.

Great, I didn't know that and I'm certainly guilty of copying and pasting that construct - and that shall stop now! Thanks!