Replace string-referencing by real methods
In the current session model many referencing is done by strings:
E.g.:
realDataItem->getItem(RealDataItem::P_NATIVE_DATA_UNITS)->setValue(units);
This is quite unsafe regarding refactoring and very inconvenient to follow references with IDEs. Also the compiler is not able to check such references.
It should be replaced by real methods, which in a first step just wrap the former call:
void RealDataItem::setNativeDataUnits(const QString& units)
{
getItem(P_NATIVE_DATA_UNITS)->setValue(units);
}
realDataItem->setNativeDataUnits(units);
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- m.puchner added GUI cleanup entry-level labels
added GUI cleanup entry-level labels
- m.puchner changed title from Replace referencing by strings to Replace string-referencing by real methods
changed title from Replace referencing by strings to Replace string-referencing by real methods
- m.puchner changed the description
changed the description
- t.knopff mentioned in merge request !58 (merged)
mentioned in merge request !58 (merged)
- t.knopff closed with merge request !58 (merged)
closed with merge request !58 (merged)
- t.knopff reopened
reopened
- t.knopff mentioned in merge request !61 (merged)
mentioned in merge request !61 (merged)
- t.knopff mentioned in merge request !63 (closed)
mentioned in merge request !63 (closed)
- t.knopff mentioned in merge request !64 (merged)
mentioned in merge request !64 (merged)
- t.knopff mentioned in merge request !67 (merged)
mentioned in merge request !67 (merged)
- t.knopff mentioned in merge request !71 (merged)
mentioned in merge request !71 (merged)
- t.knopff mentioned in merge request !102 (merged)
mentioned in merge request !102 (merged)
- t.knopff mentioned in merge request !104 (merged)
mentioned in merge request !104 (merged)
- t.knopff mentioned in merge request !106 (merged)
mentioned in merge request !106 (merged)
- t.knopff mentioned in merge request !108 (merged)
mentioned in merge request !108 (merged)
- t.knopff mentioned in merge request !110 (merged)
mentioned in merge request !110 (merged)
- t.knopff mentioned in merge request !113 (merged)
mentioned in merge request !113 (merged)
- t.knopff mentioned in merge request !120 (merged)
mentioned in merge request !120 (merged)
- t.knopff mentioned in merge request !121 (merged)
mentioned in merge request !121 (merged)
- t.knopff mentioned in merge request !122 (merged)
mentioned in merge request !122 (merged)
- t.knopff mentioned in merge request !123 (merged)
mentioned in merge request !123 (merged)
- t.knopff mentioned in merge request !124 (merged)
mentioned in merge request !124 (merged)
- t.knopff mentioned in merge request !127 (merged)
mentioned in merge request !127 (merged)
- t.knopff mentioned in merge request !128 (merged)
mentioned in merge request !128 (merged)
- t.knopff mentioned in merge request !136 (closed)
mentioned in merge request !136 (closed)
- t.knopff mentioned in merge request !137 (merged)
mentioned in merge request !137 (merged)
- t.knopff mentioned in merge request !138 (merged)
mentioned in merge request !138 (merged)
- t.knopff mentioned in merge request !139 (merged)
mentioned in merge request !139 (merged)
- t.knopff mentioned in merge request !142 (merged)
mentioned in merge request !142 (merged)
- t.knopff mentioned in merge request !148 (closed)
mentioned in merge request !148 (closed)
- t.knopff mentioned in merge request !149 (merged)
mentioned in merge request !149 (merged)
- t.knopff mentioned in merge request !151 (merged)
mentioned in merge request !151 (merged)
- t.knopff mentioned in merge request !159 (merged)
mentioned in merge request !159 (merged)
- t.knopff mentioned in merge request !168 (merged)
mentioned in merge request !168 (merged)
- t.knopff mentioned in merge request !170 (merged)
mentioned in merge request !170 (merged)
- t.knopff mentioned in merge request !172 (merged)
mentioned in merge request !172 (merged)
- t.knopff mentioned in merge request !173 (closed)
mentioned in merge request !173 (closed)
- t.knopff mentioned in merge request !175 (merged)
mentioned in merge request !175 (merged)
- t.knopff mentioned in merge request !181 (merged)
mentioned in merge request !181 (merged)
- t.knopff mentioned in merge request !213 (merged)
mentioned in merge request !213 (merged)
- m.puchner mentioned in merge request !217 (merged)
mentioned in merge request !217 (merged)
This issue was completed with merge !213 (merged)
- t.knopff closed
closed