diff --git a/modules/skybrowser/skybrowsermodule.cpp b/modules/skybrowser/skybrowsermodule.cpp index 7c765c9cfa..d1baf9f3dc 100644 --- a/modules/skybrowser/skybrowsermodule.cpp +++ b/modules/skybrowser/skybrowsermodule.cpp @@ -280,7 +280,7 @@ void SkyBrowserModule::addTargetBrowserPair(const std::string& targetId, } void SkyBrowserModule::removeTargetBrowserPair(const std::string& id) { - TargetBrowserPair* found = getPair(id); + TargetBrowserPair* found = pair(id); if (!found) { return; } @@ -299,9 +299,9 @@ void SkyBrowserModule::removeTargetBrowserPair(const std::string& id) { } void SkyBrowserModule::lookAtTarget(const std::string& id) { - TargetBrowserPair* pair = getPair(id); - if (pair) { - startRotatingCamera(pair->targetDirectionGalactic()); + TargetBrowserPair* found = pair(id); + if (found) { + startRotatingCamera(found->targetDirectionGalactic()); } } @@ -392,7 +392,7 @@ int SkyBrowserModule::nPairs() const { return static_cast(_targetsBrowsers.size()); } -TargetBrowserPair* SkyBrowserModule::getPair(const std::string& id) const { +TargetBrowserPair* SkyBrowserModule::pair(const std::string& id) const { auto it = std::find_if( _targetsBrowsers.begin(), _targetsBrowsers.end(), @@ -451,7 +451,7 @@ std::string SkyBrowserModule::wwtImageCollectionUrl() const { } void SkyBrowserModule::setSelectedBrowser(const std::string& id) { - TargetBrowserPair* found = getPair(id); + TargetBrowserPair* found = pair(id); if (found) { _selectedBrowser = id; } @@ -462,7 +462,7 @@ std::string SkyBrowserModule::selectedBrowserId() const { } std::string SkyBrowserModule::selectedTargetId() const { - TargetBrowserPair* found = getPair(_selectedBrowser); + TargetBrowserPair* found = pair(_selectedBrowser); return found ? found->targetRenderableId() : ""; } @@ -475,12 +475,12 @@ bool SkyBrowserModule::isCameraInSolarSystem() const { } bool SkyBrowserModule::isSelectedPairUsingRae() const { - TargetBrowserPair* found = getPair(_selectedBrowser); + TargetBrowserPair* found = pair(_selectedBrowser); return found ? found->isUsingRadiusAzimuthElevation() : false; } bool SkyBrowserModule::isSelectedPairFacingCamera() const { - TargetBrowserPair* found = getPair(_selectedBrowser); + TargetBrowserPair* found = pair(_selectedBrowser); return found ? found->isFacingCamera() : false; } diff --git a/modules/skybrowser/skybrowsermodule.h b/modules/skybrowser/skybrowsermodule.h index 3c4c468ae5..05d8d4def6 100644 --- a/modules/skybrowser/skybrowsermodule.h +++ b/modules/skybrowser/skybrowsermodule.h @@ -54,7 +54,7 @@ public: std::vector>& getPairs(); int nPairs() const; - TargetBrowserPair* getPair(const std::string& id) const; + TargetBrowserPair* pair(const std::string& id) const; const std::unique_ptr& getWwtDataHandler() const; std::string selectedBrowserId() const; std::string selectedTargetId() const; diff --git a/modules/skybrowser/skybrowsermodule_lua.inl b/modules/skybrowser/skybrowsermodule_lua.inl index dcf8e68903..74e50ed8a8 100644 --- a/modules/skybrowser/skybrowsermodule_lua.inl +++ b/modules/skybrowser/skybrowsermodule_lua.inl @@ -50,7 +50,7 @@ namespace { SkyBrowserModule* module = global::moduleEngine->module(); if (module->isCameraInSolarSystem()) { - TargetBrowserPair* selected = module->getPair(module->selectedBrowserId()); + TargetBrowserPair* selected = module->pair(module->selectedBrowserId()); if (selected) { const ImageData& image = module->getWwtDataHandler()->getImage(imageIndex); // Load image into browser @@ -119,7 +119,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setImageOrder(imageIndex, imageOrder); } @@ -138,7 +138,7 @@ namespace { // Load the collections here because we know that the browser can execute javascript SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->hideChromeInterface(true); pair->loadImageCollection(module->wwtImageCollectionUrl()); @@ -206,7 +206,7 @@ namespace { // Initialize browser with ID and its corresponding target LINFO("Initializing sky browser " + identifier); SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->initialize(); } @@ -377,7 +377,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setImageOpacity(imageIndex, opacity); } @@ -391,7 +391,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->centerTargetOnScreen(); } @@ -500,7 +500,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* found = module->getPair(identifier); + TargetBrowserPair* found = module->pair(identifier); if (found) { std::string browser = found->browserId(); std::string target = found->targetNodeId(); @@ -554,7 +554,7 @@ namespace { SkyBrowserModule* module = global::moduleEngine->module(); const ImageData& image = module->getWwtDataHandler()->getImage(imageIndex); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->removeSelectedImage(imageIndex); } @@ -571,7 +571,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setEquatorialAim(glm::dvec2(rightAscension, declination)); } @@ -587,7 +587,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setVerticalFov(verticalFieldOfView); } @@ -601,7 +601,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setVerticalFovWithScroll(scroll); } @@ -617,7 +617,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setBorderColor(glm::ivec3(red, green, blue)); } @@ -632,7 +632,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->setBrowserRatio(ratio); } @@ -650,7 +650,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->browser()->addDisplayCopy(position, numberOfCopies); } @@ -663,7 +663,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->browser()->removeDisplayCopy(); } @@ -676,7 +676,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { pair->startFinetuningTarget(); } @@ -694,7 +694,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { glm::vec2 startScreenSpace = skybrowser::pixelToScreenSpace2d(startPosition); glm::vec2 endScreenSpace = skybrowser::pixelToScreenSpace2d(endPosition); @@ -711,7 +711,7 @@ namespace { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); if (pair) { LINFO("Image collection is loaded in Screen Space Sky Browser " + identifier); pair->setImageCollectionIsLoaded(true); @@ -748,7 +748,7 @@ namespace { [[codegen::luawrap]] void pointSpaceCraft(std::string identifier) { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); glm::dvec2 equatorial = pair->targetDirectionEquatorial(); global::eventEngine->publishEvent( equatorial.x, @@ -763,7 +763,7 @@ namespace { [[codegen::luawrap]] void stopAnimations(std::string identifier) { using namespace openspace; SkyBrowserModule* module = global::moduleEngine->module(); - TargetBrowserPair* pair = module->getPair(identifier); + TargetBrowserPair* pair = module->pair(identifier); pair->stopAnimations(); } diff --git a/modules/skybrowser/src/screenspaceskybrowser.cpp b/modules/skybrowser/src/screenspaceskybrowser.cpp index d434a6b2c3..144730e82d 100644 --- a/modules/skybrowser/src/screenspaceskybrowser.cpp +++ b/modules/skybrowser/src/screenspaceskybrowser.cpp @@ -146,7 +146,7 @@ ScreenSpaceSkyBrowser::ScreenSpaceSkyBrowser(const ghoul::Dictionary& dictionary ScreenSpaceSkyBrowser::~ScreenSpaceSkyBrowser() { SkyBrowserModule* module = global::moduleEngine->module(); - if (module && module->getPair(identifier())) { + if (module && module->pair(identifier())) { module->removeTargetBrowserPair(identifier()); } }