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

On search page, keyword hints don't seem to work #26

Closed
torrin47 opened this issue Aug 17, 2016 · 7 comments
Closed

On search page, keyword hints don't seem to work #26

torrin47 opened this issue Aug 17, 2016 · 7 comments
Assignees

Comments

@torrin47
Copy link
Contributor

Clicking on keyword hints fires this URL:
https://edg-staging.epa.gov/metadata/rest/find/document?f=searchpage&searchText=keywords%3A%22soils%20%26%20land%22
which results in a null response. Log file attached.
metadata.2016-08-17.zip

@Baohong
Copy link
Contributor

Baohong commented Sep 27, 2016

Currently the gpt library on staging is of version 1.2.7. Do I need to change it to gpt1.2.6 and revert all java class under \WEB-INF\classes\com\ to original java class (github version) based on 1.2.6 and then debug the search page? Please confirm!

@torrin47
Copy link
Contributor Author

No, everything works fine when we revert to 1.2.6, I'd prefer moving in the
other direction - getting it working under 1.2.7.

On Tue, Sep 27, 2016 at 5:25 AM, Baohong notifications@github.com wrote:

Currently the gpt library on staging is of version 1.2.7. Do I need to
change it to gpt1.2.6 and revert all java class under \WEB-INF\classes\com
to original java class (github version) based on 1.2.6 and then debug the
search page? Please confirm!


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#26 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACB0vMv7k2Qg9IQRJ1CSuwlZJ4MEWuOEks5quQtRgaJpZM4Jm6_U
.

Torrin Hultgren
Senior Manager, Geospatial Solutions
Innovate!, Inc.
O: (703) 922-9090 x737

http://www.innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security,
Business Consulting, Project Management and Executive Coaching
solutions to our government and private sector clients. Check us out
at www.innovateteam.com.

@Baohong
Copy link
Contributor

Baohong commented Sep 27, 2016

I think the search page problem is caused by the java code while upgrading to 1.2.7 --- see my Email I sent yesterday. Suma is working on fixing the upgrading errors of java file. After she fix all the java files, if the search page problem still exist, I will debug on that, is that Ok?

@torrin47
Copy link
Contributor Author

Sounds good, much appreciated.

On Tue, Sep 27, 2016 at 8:22 AM, Baohong notifications@github.com wrote:

I think the search page problem is caused by the java code while upgrading
to 1.2.7 --- see my Email I sent yesterday. Suma is working on fixing the
upgrading errors of java file. After she fix all the java files, if the
search page problem still exist, I will debug on that, is that Ok?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#26 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACB0vFmhroVjfAoBBq5fOcvPjoOLF4Eqks5quTTIgaJpZM4Jm6_U
.

Torrin Hultgren
Senior Manager, Geospatial Solutions
Innovate!, Inc.
O: (703) 922-9090 x737

http://www.innovateteam.com

Innovate!, Inc. is a Virginia-based 8(a), Hispanic- and woman-owned
business delivering Geospatial, Information Technology, Security,
Business Consulting, Project Management and Executive Coaching
solutions to our government and private sector clients. Check us out
at www.innovateteam.com.

Saisuma004 pushed a commit that referenced this issue Oct 4, 2016
@Baohong
Copy link
Contributor

Baohong commented Oct 4, 2016

One step further in upgrading/debugging java:

In WEB-INF\classes\com\esri\gpt\control\georss\RestQueryServlet.java, parameter "context" should be removed from following code: GetRecordsGenerator grg = new GetRecordsGenerator(context);
This modification will fix the error "Unsupported operation: GetRecords", and will get the search result.

There are several definitions for function GetRecordsGenerator: with parameter or without parameter. gpt1.2.6 is using GetRecordsGenerator(context), but gpt1.2.7 is using GetRecordsGenerator(). If we still use GetRecordsGenerator(context) which generates no compilation error, our CSW request will be "http://www.opengis.net/cat/csw/3.0". While namespace in code should be "http://www.opengis.net/cat/csw/2.0.2". This mismatch results in "Unsupported operation: GetRecords" and no search result could be returned.

This file will be updated to Suma. There is one more modification in file based on 1.2.7, but maybe not related to search issue.

However, even after we get search result, there are problem in rendering result to search page. I will continue to debug this error.

@Baohong
Copy link
Contributor

Baohong commented Oct 7, 2016

Search hint is now working on staging.

After fixing all java files. There is rendering problem on search page. This issue is fixed by deleting some of .jar files in folder lib.

I compared lib folder with \geoportal-server-1.2.7\geoportal\lib and found some .jar files of higher version than that of \geoportal-server-1.2.7\geoportal\lib. By deleting some of the extra .jar files. The rendering bug disappear. Torrin and Catherine, please check folder metadata\WEB-INF\lib to see if any .jar files need to be put back. The original lib folder is renamed as lib_old.

@torrin47
Copy link
Contributor Author

torrin47 commented Oct 7, 2016

Awesome, looking good to me, I'm not seeing any issues, and the search term logging is working. Woohoo! I'll ask folks to pound on it for a few days next week for good measure, but so far can't see any reason not to get this into production!

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

No branches or pull requests

2 participants