#4675 Remove unused code #10442
Open
#4675 Remove unused code #10442
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
13 tasks
|
Thank you @allyoucanmap |
5 tasks
offtherailz
requested changes
Jul 10, 2024
There was a problem hiding this comment.
I'm not sure about the following:
-
WFS3 support.
web/client/api/WFS3.jsweb/client/api/__tests__/WFS3-test.jsand soweb/client/components/map/openlayers/__tests__/Layer-test.jsxweb/client/components/map/openlayers/plugins/WFS3Layer.jsweb/client/components/map/openlayers/plugins/index.js
Why? -
I think that we can remove also these dependencies from
package.json:-
"ag-grid-community": "20.2.0": used only inweb/client/components/style/ColorMapGrid.jsx -
"ag-grid-react": "20.2.0": same as above -
"colorbrewer": "1.0.0": used only inweb/client/components/style/EqualIntervalComponents/ExtendColorBrewer.js
-
About dependencies, other things to take in consideration are:
- Uniform turf versions
- I noticed also
react-confirm-button, not in this PR but not used for a long time - From my undestanding also
wkt-parser(documented also inmoduleFederationandcustom-dependencies.mdcan be removed.wellknowncan be replaced with our toGeoJSon and toWKT functions in MapStore. Maybe this last can be postponed to a second time.
About files appended:
Can be removed
"MapStore2/web/client/actions/importer.js",
"MapStore2/web/client/reducers/importer.js",
"MapStore2/web/client/plugins/manager/Importer.jsx",
"MapStore2/web/client/components/manager/importer/BreadCrumb.jsx",
"MapStore2/web/client/components/manager/importer/Import.jsx",
"MapStore2/web/client/components/manager/importer/Importer.jsx",
"MapStore2/web/client/components/manager/importer/ImportsGrid.jsx",
"MapStore2/web/client/components/manager/importer/Workspace.jsx",
"MapStore2/web/client/components/manager/importer/Layer.jsx",
"MapStore2/web/client/components/manager/importer/Task.jsx",
"MapStore2/web/client/components/manager/importer/TaskProgress.jsx",
"MapStore2/web/client/components/manager/importer/Transform.jsx",
"MapStore2/web/client/components/manager/importer/TransformsGrid.jsx",
"MapStore2/web/client/components/manager/importer/style/importer.css",
"MapStore2/web/client/components/manager/importer/transforms/GdalAddoTransform.jsx",
"MapStore2/web/client/components/manager/importer/transforms/GdalTranslateTransform.jsx",
"MapStore2/web/client/components/manager/importer/transforms/GdalWarpTransform.jsx",
"MapStore2/web/client/components/manager/importer/transforms/index.js",
Belong to the importer that is not maintained and used anymore.
- actions, reducers and components could be removed.
"MapStore2/web/client/utils/AgentUtils.js",
Can be removed, used only be elements removed.
"MapStore2/web/client/actions/share.js",
"MapStore2/web/client/actions/tasks.js",
"MapStore2/web/client/reducers/tasks.js",
I don't Know probably can be removed.
"MapStore2/web/client/api/MapConfigDAO.js",
Very old file, not used can be removed
"MapStore2/web/client/containers/HolyGrail.jsx",
Alternative layout container for MapViewer/Pages, not used anymore for a long time.
BorderLayout is more than enough, can be removed.
"MapStore2/web/client/observables/ogcDimension.js",
Doesn't seems to be used. Probably it has been removed from a refactor.
Can be removed
"MapStore2/web/client/components/mapcontrols/search/geocoding/NominatimResult.jsx",
"MapStore2/web/client/components/mapcontrols/search/geocoding/NominatimResultList.jsx",
Probably old implementation, can be removed
"MapStore2/web/client/components/maps/MapItem.jsx",
"MapStore2/web/client/components/maps/MapList.jsx",
These are probably old, replaced by generic GridCards, can be removed.
Maybe can be removed
"MapStore2/web/client/api/geoserver/Importer.js",
"MapStore2/web/client/api/geoserver/Workspaces.js",
- Used by importer, maybe APIs could still make sense. Anyway they are not used.
Maybe Keep
"MapStore2/web/client/utils/ogc/WFST/index.js",
"MapStore2/web/client/utils/ogc/WPS/base.js",
"MapStore2/web/client/utils/ogc/WPS/index.js",
"MapStore2/web/client/utils/ogc/WPS/RequestBuilder.js",
"MapStore2/web/client/utils/ogc/WFST/v1_1_0/delete.js",
"MapStore2/web/client/utils/ogc/WFST/v1_1_0/index.js",
"MapStore2/web/client/utils/ogc/WFST/v1_1_0/insert.js",
"MapStore2/web/client/utils/ogc/WFST/v1_1_0/transaction.js",
"MapStore2/web/client/utils/ogc/WFST/v1_1_0/update.js",
All these utils are maybe should be keep. Not sure why they are orphan.
"MapStore2/web/client/plugins/print/Graticule.jsx",
"MapStore2/web/client/plugins/print/Scale.jsx",
We have to make sure they are not plugins of print.
Not processed
I didn't have time to process all these files.
We need to spend more time on them.
"MapStore2/web/client/reducers/selection.js",
"MapStore2/web/client/selectors/cswtocatalog.js",
"MapStore2/web/client/api/usersession/serverbackup.js",
"MapStore2/web/client/components/app/StandardAppComponent.jsx",
"MapStore2/web/client/components/file/FileUploader.jsx",
"MapStore2/web/client/components/help/HelpToggleBtn.jsx",
"MapStore2/web/client/components/I18N/LangSelector.jsx",
"MapStore2/web/client/components/misc/DockablePanel.jsx",
"MapStore2/web/client/components/misc/LinkToPage.jsx",
"MapStore2/web/client/components/misc/ResizableGrid.jsx",
"MapStore2/web/client/components/print/PrintSubmit.jsx",
"MapStore2/web/client/components/security/PermissionEditor.jsx",
"MapStore2/web/client/components/toolbar/MapToolbar.jsx",
"MapStore2/web/client/plugins/dashboard/Editor.jsx",
"MapStore2/web/client/plugins/dashboard/SaveDialog.jsx",
"MapStore2/web/client/plugins/searchservicesconfig/ToggleButton.jsx",
"MapStore2/web/client/product/pages/Importer.jsx",
"MapStore2/web/client/components/data/identify/DefaultHeader.jsx",
"MapStore2/web/client/components/data/query/AutocompleteField.jsx",
"MapStore2/web/client/components/data/query/SimpleFilterField.jsx",
"MapStore2/web/client/components/manager/users/GroupManager.jsx",
"MapStore2/web/client/components/maps/enhancers/mapCatalog.js",
"MapStore2/web/client/components/maps/forms/Metadata.jsx",
"MapStore2/web/client/components/mediaEditor/enhancers/withFilter.js",
"MapStore2/web/client/components/misc/datetimepicker/Time.js",
"MapStore2/web/client/components/misc/enhancers/debounce.js",
"MapStore2/web/client/components/misc/enhancers/withPropsStream.js",
"MapStore2/web/client/components/misc/sidebar/index.js",
"MapStore2/web/client/components/misc/sidebar/Sidebar.jsx",
"MapStore2/web/client/components/misc/sidebar/SidebarHeader.jsx",
"MapStore2/web/client/components/TOC/fragments/RefreshLayers.jsx",
"MapStore2/web/client/components/widgets/builder/WidgetBuilder.jsx",
"MapStore2/web/client/components/widgets/enhancers/dataHolder.js",
"MapStore2/web/client/product/assets/css/home.css",
"MapStore2/web/client/product/components/home/Brand.jsx",
"MapStore2/web/client/product/components/home/Description.jsx",
"MapStore2/web/client/product/components/home/Footer.jsx",
"MapStore2/web/client/product/components/home/Fork.jsx",
"MapStore2/web/client/product/components/home/Language.jsx",
"MapStore2/web/client/product/components/home/Logo.jsx",
"MapStore2/web/client/product/components/viewer/Home.jsx",
"MapStore2/web/client/product/plugins/footer/footer.css",
"MapStore2/web/client/product/plugins/homedescription/homedescription.css",
"MapStore2/web/client/components/data/featuregrid/dialog/index.js",
"MapStore2/web/client/components/data/featuregrid/renderers/cell.css",
"MapStore2/web/client/components/data/identify/css/identify.css",
"MapStore2/web/client/components/map/openlayers/plugins/MarkerLayer.js",
"MapStore2/web/client/components/maps/modals/css/modals.css",
"MapStore2/web/client/components/misc/spinners/InlineSpinner/InlineSpinner.jsx",
"MapStore2/web/client/components/TOC/fragments/settings/FeatureInfoFormat.jsx",
"MapStore2/web/client/plugins/StreetView/components/CyclomediaView/index.jsx",
"MapStore2/web/client/plugins/StreetView/components/MapillaryView/__test__/MapillaryView-test.jsx"
WFS3 was an experimental specification that it was replaced later on by the OGC Api features, in addition is a partial implementation without support for 3D.
Removed
Removed @tdipisa I think this other tasks highlighted in the review should be solved in separated issues
Can be removed sectionThe content described in the Maybe can be removed and Not processed sectionI review the missing files to check, below the list of removed files with a small descrition on the reason
Remaining unused filesWe still have the following unused files after removing all the above and running again the
|
I agree with the following with separated issues. As for the following:
I would like to wait for the @offtherailz review. Probably the most part of pending things can be totally removed. §Let's discuss together after the review @offtherailz @allyoucanmap. |
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR removes plugins and related dependencies unused and deprecated. The current plugins has been removed:
FeatureGridreplaced byFeatureEditorMeasurePanelreplaced byMeasureMeasureResultsreplaced byMeasureRasterStylerreplaced byStyleEditorShapeFilereplaced byMapImportStylerreplaced byStyleEditorVectorStylerreplaced byStyleEditorAdditional code changes:
Annotationsdeprecated code has been removed because replaced by the new refactoringPlease check if the PR fulfills these requirements
What kind of change does this PR introduce? (check one with "x", remove the others)
Issue
What is the current behavior?
#4675
What is the new behavior?
Removed unused code and deprecated components
Breaking change
Does this PR introduce a breaking change? (check one with "x", remove the other)
Other useful information
The
webpack-deadcode-pluginlib has been used to check which files are not used by mapstore, there are still the following components that should be checked: