+ introduce proper enum for property status to replace plain integers
This commit is contained in:
@@ -86,10 +86,10 @@ void ViewProviderDocumentObject::onChanged(const App::Property* prop)
|
||||
}
|
||||
else if (prop == &Visibility) {
|
||||
// use this bit to check whether show() or hide() must be called
|
||||
if (Visibility.StatusBits.test(8) == false) {
|
||||
Visibility.StatusBits.set(8);
|
||||
if (Visibility.testStatus(App::Property::User2) == false) {
|
||||
Visibility.setStatus(App::Property::User2, true);
|
||||
Visibility.getValue() ? show() : hide();
|
||||
Visibility.StatusBits.reset(8);
|
||||
Visibility.setStatus(App::Property::User2, false);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -99,10 +99,10 @@ void ViewProviderDocumentObject::onChanged(const App::Property* prop)
|
||||
void ViewProviderDocumentObject::hide(void)
|
||||
{
|
||||
// use this bit to check whether 'Visibility' must be adjusted
|
||||
if (Visibility.StatusBits.test(8) == false) {
|
||||
Visibility.StatusBits.set(8);
|
||||
if (Visibility.testStatus(App::Property::User2) == false) {
|
||||
Visibility.setStatus(App::Property::User2, true);
|
||||
Visibility.setValue(false);
|
||||
Visibility.StatusBits.reset(8);
|
||||
Visibility.setStatus(App::Property::User2, false);
|
||||
}
|
||||
ViewProvider::hide();
|
||||
}
|
||||
@@ -110,10 +110,10 @@ void ViewProviderDocumentObject::hide(void)
|
||||
void ViewProviderDocumentObject::show(void)
|
||||
{
|
||||
// use this bit to check whether 'Visibility' must be adjusted
|
||||
if (Visibility.StatusBits.test(8) == false) {
|
||||
Visibility.StatusBits.set(8);
|
||||
if (Visibility.testStatus(App::Property::User2) == false) {
|
||||
Visibility.setStatus(App::Property::User2, true);
|
||||
Visibility.setValue(true);
|
||||
Visibility.StatusBits.reset(8);
|
||||
Visibility.setStatus(App::Property::User2, false);
|
||||
}
|
||||
ViewProvider::show();
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user