[GeoNetwork-devel] DB script questions

I did some recent work on the DB scripts so that the migration script would
perform the same functionality as the creation scripts. I'm left with 2
issues

1 - When the spring security was added, a new user called "developer" was
also added at the end of the following file.
web/src/main/webapp/WEB-INF/classes/setup/sql/data/data-db-default.sql

This new "developer" user was not added in the migration scripts.

What is the purpose of this user? Should it be added to the migration script
as well? If not should it be removed from the data-db-default.sql?

2 - As part of commit e92c466c560041021729042091e567dc9f2f3a54 (merge the
improved search based on lucene 4.1.) The following line was removed from
web/src/main/webapp/WEB-INF/classes/setup/sql/data/data-db-default.sql

   INSERT INTO Settings VALUES (955,952,'ignored','true');

Should there be a "delete" in the migration scripts - Will this setting
cause any issues?

Thank you.

--
View this message in context: http://osgeo-org.1560.x6.nabble.com/DB-script-questions-tp5056210.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

On Mon, May 27, 2013 at 7:20 PM, ianwallen <ianwallen@anonymised.com> wrote:

I did some recent work on the DB scripts so that the migration script would
perform the same functionality as the creation scripts. I'm left with 2
issues

1 - When the spring security was added, a new user called "developer" was
also added at the end of the following file.
web/src/main/webapp/WEB-INF/classes/setup/sql/data/data-db-default.sql

This new "developer" user was not added in the migration scripts.

What is the purpose of this user? Should it be added to the migration
script
as well? If not should it be removed from the data-db-default.sql?

I hadn't realized I had committed the developer user. I was playing with
an idea at one point that there could be a developer user that can access
extra analytics that admins can't because the admins that I work with are
not as technical as I am. But I decided later that it is not useful. That
should be removed.

2 - As part of commit e92c466c560041021729042091e567dc9f2f3a54 (merge the
improved search based on lucene 4.1.) The following line was removed from
web/src/main/webapp/WEB-INF/classes/setup/sql/data/data-db-default.sql

   INSERT INTO Settings VALUES (955,952,'ignored','true');

Should there be a "delete" in the migration scripts - Will this setting
cause any issues?

This one I don't know about...

--
View this message in context:
http://osgeo-org.1560.x6.nabble.com/DB-script-questions-tp5056210.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

------------------------------------------------------------------------------
Try New Relic Now & We'll Send You this Cool Shirt
New Relic is the only SaaS-based application performance monitoring service
that delivers powerful full stack analytics. Optimize and monitor your
browser, app, & servers with just a few lines of code. Try New Relic
and get this awesome Nerd Life shirt! http://p.sf.net/sfu/newrelic_d2d_may
_______________________________________________
GeoNetwork-devel mailing list
GeoNetwork-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geonetwork-devel
GeoNetwork OpenSource is maintained at
http://sourceforge.net/projects/geonetwork

Jesse

I just submitted pull request #96
<https://github.com/geonetwork/core-geonetwork/pull/96&gt; to remove the
developer user.

As for the lucene 4.1 issue. It look like this change was committed by you
(if you are like me you have a short memory ). I just looked at the
committed changes and it seems that all reference to "RequestedLanguage
Ignore" have been removed so leaving in the settings does not seem like it
will cause any issues. So I will simply leave it as is and not bother making
any change.

Thank you.

--
View this message in context: http://osgeo-org.1560.x6.nabble.com/DB-script-questions-tp5056210p5056383.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

On Tue, May 28, 2013 at 1:08 PM, ianwallen <ianwallen@anonymised.com> wrote:

Jesse

I just submitted pull request #96
<https://github.com/geonetwork/core-geonetwork/pull/96&gt; to remove the
developer user.

Thanks, I will review and merge it.

As for the lucene 4.1 issue. It look like this change was committed by you
(if you are like me you have a short memory ). I just looked at the
committed changes and it seems that all reference to "RequestedLanguage
Ignore" have been removed so leaving in the settings does not seem like it
will cause any issues. So I will simply leave it as is and not bother
making
any change.

I seem to remember something about this setting. I think it might be used
for tests at least or something. Maybe I should check the docs and see if
it is mentioned there as part of the configuration docs... Needs to be
investigated a little.