[GeoNetwork-devel] added sort language back to ui - causing issues

Jesse

In the 2.8 branch, you reverted some changes for commit
"3df27d9a77ecfcf3debc6e7166f46a02ef873ccc" - "added sort language back to
ui" which removed the following line from
"web/src/main/webapp/loc/eng/xml/config.xml"

<requestedlanguagesorted>Search results in requested language sorted on
top</requestedlanguagesorted>

This is causing empty text to show up next to the the checkbox on the
multilingual setting in the configuration section.

Also, since this changes the look of the interface. Once it is corrected, it
look like a new screenshot will be required for the documentation.
eng/users/source\admin/configuration/web-config-options-2.png

--
View this message in context: http://osgeo-org.1560.n6.nabble.com/added-sort-language-back-to-ui-causing-issues-tp5026410.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

I will take a look. The reason I added this back is because a fix back in december caused the integration tests to fail because they require the sorted field. I don’t want a hidden configuration parameter so I added it to the UI.

···

On Wed, Jan 9, 2013 at 4:37 PM, ianwallen <ianwallen@anonymised.com> wrote:

Jesse

In the 2.8 branch, you reverted some changes for commit
“3df27d9a77ecfcf3debc6e7166f46a02ef873ccc” - “added sort language back to
ui” which removed the following line from
“web/src/main/webapp/loc/eng/xml/config.xml”

Search results in requested language sorted on
top

This is causing empty text to show up next to the the checkbox on the
multilingual setting in the configuration section.

Also, since this changes the look of the interface. Once it is corrected, it
look like a new screenshot will be required for the documentation.
eng/users/source\admin/configuration/web-config-options-2.png


View this message in context: http://osgeo-org.1560.n6.nabble.com/added-sort-language-back-to-ui-causing-issues-tp5026410.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.


Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only – learn more at:
http://p.sf.net/sfu/learnmore_122612


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

Just and FYI

The English config seems to be the only one where you removed 2 lines - So
you may have accidentally removed an extra line...

--
View this message in context: http://osgeo-org.1560.n6.nabble.com/added-sort-language-back-to-ui-causing-issues-tp5026410p5026430.html
Sent from the GeoNetwork developer mailing list archive at Nabble.com.

Yes that is what I discovered I have added it back locally and am now looking to update image

···

On Wed, Jan 9, 2013 at 5:25 PM, ianwallen <ianwallen@anonymised.com> wrote:

Jesse

Just and FYI

The English config seems to be the only one where you removed 2 lines - So
you may have accidentally removed an extra line…


View this message in context: http://osgeo-org.1560.n6.nabble.com/added-sort-language-back-to-ui-causing-issues-tp5026410p5026430.html

Sent from the GeoNetwork developer mailing list archive at Nabble.com.


Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only – learn more at:
http://p.sf.net/sfu/learnmore_122612


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

Done.

···

On Wed, Jan 9, 2013 at 5:26 PM, Jesse Eichar <jesse.eichar@anonymised.com> wrote:

Yes that is what I discovered I have added it back locally and am now looking to update image

On Wed, Jan 9, 2013 at 5:25 PM, ianwallen <ianwallen@anonymised.com> wrote:

Jesse

Just and FYI

The English config seems to be the only one where you removed 2 lines - So
you may have accidentally removed an extra line…


View this message in context: http://osgeo-org.1560.n6.nabble.com/added-sort-language-back-to-ui-causing-issues-tp5026410p5026430.html

Sent from the GeoNetwork developer mailing list archive at Nabble.com.


Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only – learn more at:
http://p.sf.net/sfu/learnmore_122612


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