View Issue Details

IDProjectCategoryView StatusLast Update
0012005ScribusStory Editor / Text Framespublic2016-01-20 21:25
Reporterale Assigned To 
PrioritynormalSeverityminorReproducibilityN/A
Status newResolutionopen 
Summary0012005: GetText2?
Descriptionwith the xtg text loader plugin, a GetText2 function has been introduced.

on the one side, the commit message does not specify why the new function has been introduced in gtgettext.cpp, if this method should be used in place of the GetText used by all the other text loading plugins.

on the second side, when you load text content through a plugin that does not define it, you get a "GetText2" missing warning. not a nice thing.

and on the third side, it's not clear to me what the improvement of GetText2 is. at a first sight, it looks like, passing textItem is a better option, but -- at the end -- the plugin only uses textItem to extract gWriter from there, which is the parameter of GetText. (ok, i guess that if it could also avoid to get the current document through a static call...).

so, can somebody in the team have a look at it, and:
- add some documentation in the code explaining why GetText2 is there, or
- remove GetText2 and patch the xtg loader to use the good old (ok, old.) GetText?
TagsNo tags attached.
Patch

Activities

Kunda

2016-01-20 21:25

updater   ~0038309

discussion?

Issue History

Date Modified Username Field Change
2014-01-22 13:39 ale New Issue
2015-09-17 20:08 Kunda Category Story Editor / Text Frames => Story Ed/Txt Frames
2015-09-17 20:12 Kunda Category Story Ed/Txt Frames => Story Editor / Text Frames
2016-01-20 21:25 Kunda Note Added: 0038309