Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#678] fixes NPE using SearchView #680

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

fgdrf
Copy link
Contributor

@fgdrf fgdrf commented Feb 7, 2022

No description provided.

Change-Id: I8ee2cbc36a1cca3c559b75fb6c2858acb8059d3d
Signed-off-by: Frank Gasdorf <fgdrf@users.sourceforge.net>
Change-Id: I6c9281cbc82ff319a4e7f5d5d01dd252716ad5b3
Signed-off-by: Frank Gasdorf <fgdrf@users.sourceforge.net>
Change-Id: I1d7e4a3df7d6186875f29af21fbb23c4f17d01e3
Signed-off-by: Frank Gasdorf <fgdrf@users.sourceforge.net>
Change-Id: If13cfb5c1bc3cc5bb9b4a4038d4df1a970f6d93e
Signed-off-by: Frank Gasdorf <fgdrf@users.sourceforge.net>
@sschulz92
Copy link
Contributor

In the description you may refer to the issue to link them :)
#678

resources, if the namespace of the type is null

Change-Id: I255a9f14a3999e7286570dcf50ed0014b524d458
Signed-off-by: Frank Gasdorf <fgdrf@users.sourceforge.net>
@fgdrf fgdrf changed the title fixes NPE reported in issue 678 [#678] fixes NPE using SearchView Feb 7, 2022
@@ -199,7 +199,7 @@ protected DB2GeoResourceInfo createInfo(IProgressMonitor monitor) throws IOExcep

icon = Glyph.icon(ft);
keywords = new String[] { "db2", //$NON-NLS-1$
ft.getName().getLocalPart(), ft.getName().getNamespaceURI() };
ft.getName().getLocalPart(), ft.getName().getURI() };
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be similar to shp implementation. check for namespace, if not null, add it otherwise skip it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NullPointerException in ShpGeoResourceInfo while executing search in SearchView
2 participants