2012-01-26  Murray Cumming  <murrayc@murrayc.com>

	Use the ?locale= query param instead of the &lang= token param. 

	* src/main/java/org/glom/web/client/place/ListPlace.java

	* src/main/java/org/glom/web/client/place/DetailsPlace.java

	* src/main/java/org/glom/web/client/place/HasSelectableTablePlace.java

	Remove the lang token key and value.

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java:

	When the user selects a different locale from the chooser, use 

	Window.Location.assign() to change the URL, which then causes a reload.

	* src/main/java/org/glom/web/client/Utils.java: Added getCurrentLocaleID().

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java

	* src/main/java/org/glom/web/client/activity/DocumentSelectionActivity.java

	* src/main/java/org/glom/web/client/activity/ListActivity.java

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	* src/main/java/org/glom/web/client/ui/DocumentSelectionView.java

	* src/main/java/org/glom/web/client/ui/DocumentSelectionViewImpl.java:

	* src/main/java/org/glom/web/client/ui/ListView.java:

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	Remove localeID member variables and method/constructor parameters, instead

	using Utils.getCurrentLocaleID() when we need a localID to pass to 

	OnlineGlomService.

2012-01-26  Murray Cumming  <murrayc@murrayc.com>

	Internationalize the UI strings.

	* pom.xml: gwt-maven-plugin: Add the i18n goal and specify a 

	<i18nConstantsBundle>, removing the unused <i18nMessagesBundle>.

	* src/main/resources/org/glom/web/client/Messages.properties: Remove this

	because it is unused. Messages are apparently strings that can have 

	parameters, but we do not need that yet, so Contants will be enough for now.

	* src/main/java/org/glom/web/OnlineGlom.gwt.xml: Add extend-property lines

	to say that we support the en and de locales.

	* src/main/resources/org/glom/web/client/ui/OnlineGlomConstants.properties:

	The original English strings.

	* src/main/resources/org/glom/web/client/ui/OnlineGlomConstants_de.properties:

	Some German translations of the English strings.

	The i18n goal then uses the .properties file to generate an 

	OnlineGlomConstants.java file in target/ and somehow GWT.create() magically

	returns an implementation that returns the translated strings.

	The documentation suggests putting these in src/java/*/client/, but it seems

	best to put it in src/resources/*/client/.

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java:

	Instantiate OnlineGlomConstants via GWT.create() and use it to get the strings

	instead of hard-coding them.

	Note that we cannot import OnlineGlomConstants because it does not exist yet,

	but that does not seem to stop the build, though it confuses Eclipse.

	You can see the translated string by adding ?locale=de to the URL, like so:

	http://127.0.0.1:8888/OnlineGlom.html?gwt.codesvr=127.0.0.1:9997?locale=de#list:document=film_manager

2012-01-24  Murray Cumming  <murrayc@murrayc.com>

	Improve null/empty String checks. 

	* pom.xml: Add a dependency on commons-lang, to use

	org.apache.commons.lang.StringUtils.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

	Use StringUtils.isEmpty().

	* src/main/java/org/glom/web/client/StringUtils.java: Add a tiny

	StringUtils class with a static isEmpty() function because we 

	cannot use org.apache.commons.lang.StringUtils in client-side

	GWT code because it (apparently) cannot be compiled to javascript.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java

	* src/main/java/org/glom/web/client/activity/ListActivity.java

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity.java

	* src/main/java/org/glom/web/client/place/DetailsPlace.java

	* src/main/java/org/glom/web/client/place/HasSelectableTablePlace.java

	* src/main/java/org/glom/web/client/place/ListPlace.java

	* src/main/java/org/glom/web/client/ui/cell/NumericCell.java

	* src/main/java/org/glom/web/client/ui/cell/TextCell.java

	* src/main/java/org/glom/web/client/ui/details/DetailsCell.java

	* src/main/java/org/glom/web/client/ui/details/Group.java

	* src/main/java/org/glom/web/client/ui/details/Notebook.java: Use 

	our StringUtils.isEmpty() function.

2012-01-24  Murray Cumming  <murrayc@murrayc.com>

	Update to the latest java-libglom API.

	* pom.xml: Require java-libglom 1.21.4.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java

	getDocumentInfo(), getListViewLayoutGroup():

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java

	getDocuments():

	* src/main/java/org/glom/web/server/database/DBAccess.java

	getFieldsToShowForSQLQueryAddGroup(),

	getPrimaryKeyLayoutItemField(): Replace get_database_title()

	with either get_database_title_original() or 

	get_database_title(localeID).

2012-01-24  Murray Cumming  <murrayc@murrayc.com>

	ConfiguredDocument: Avoid a null pointer exception.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java

	Initialize localeID to "" to avoid returning a null String which 

	causes a crash in java-libglom's swing-generated code.

2012-01-23  Murray Cumming  <murrayc@murrayc.com>

	Some simple renaming.

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java

	* src/main/webapp/style.css: Rename, tableChooser to tablesChooser. Likewise

	for localeChooser. This seems more appropriate and is less ambiguous 

	particularly in the .css file.

2012-01-23  Murray Cumming  <murrayc@murrayc.com>

	ConfiguredDocument: Rename the localedID private member variable.

2012-01-23  Murray Cumming  <murrayc@murrayc.com>

	Adapt to the latest java-libglom API from git master.

	* src/main/java/org/glom/web/server/database/ListDBAccess.java:

	libglom now uses only Vector instead of List, which uses add() instead of

	addLast(). 

2012-01-20  Murray Cumming  <murrayc@murrayc.com>

	Build a source tarball with mvn assembly:single

	* assembly.xml: Add this file.

	* pom.xml: Use the maven-assembly-plugin and tell it to use 

	our assembly.xml file.

2012-01-19  Murray Cumming  <murrayc@murrayc.com>

	OnlineGlomServiceImpl: Get .glom files recursively.

	* pom.xml: Depend on commons-io from org.apache.commons.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java

	init(): Use org.apache.commons.io.FileUtils.listFiles() to get the 

	files recursively, and with the easier filter for the extension.

	Use org.apache.commons.io.FilenameUtils.removeExtension() to 

	simplify that code too.

2012-01-19  Murray Cumming  <murrayc@murrayc.com>

	README: Mention that you must install java-libglom packages separately.

	But then it works, because java-libglom is now in the central maven 

	repository.

2012-01-18  Murray Cumming  <murrayc@murrayc.com>

	locales drop-down: Show the correct selected locale when the URL changes.

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	.java: setPlace(): Move some code into fillView().

2012-01-18  Murray Cumming  <murrayc@murrayc.com>

	locales drop-down: Do not lose the primary key.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java

	start(): onLocaleChange(): Pass the current primary key value, 

	instead of an empty value.

2012-01-18  Murray Cumming  <murrayc@murrayc.com>

	locales drop-down: Do not lose the drop-down selection.

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	.java (TableSelectionActivity.fillView): Set the selected locale

	after changing the drop-down items (though we do not really need

	to change them just because the locale changes.)

2012-01-18  Murray Cumming  <murrayc@murrayc.com>

	locales drop-down: Change the tables list when this changes.

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	.java: TableSelectionActivity.start(): Move the async table titles

	retrieval into a private fillView() method and also call this when 

	the chosen locale changes.

	Note that the document title is not actually translatable yet, but

	that is a problem that I should fix soon in libglom.

2012-01-18  Murray Cumming  <murrayc@murrayc.com>

	Improve the placement of the locales drop-down.

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java

	Put the title and locales drop-down in a div (gwt.FlowTable).

	* src/main/webapp/style.css: Add magic css properties to make this work.

	Also remove the left margin from the title so that it lines up with the 

	headerbox below it.

2012-01-18  Murray Cumming  <murrayc@murrayc.com>

	locales selector: Show human-readable locale titles.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java

	getDocumentInfo(): Use java.util.Locale to show a real title of 

	each locale, in the locale's own language.

2012-01-17  Murray Cumming  <murrayc@murrayc.com>

	Add a language/locale selector drop-down.

	* src/main/java/org/glom/web/shared/DocumentInfo.java:

	Add getLocaleIDs(), setLocaleIDs(), getLocaleTitles(), setLocaleTitles().

	* /src/main/java/org/glom/web/server/ConfiguredDocument.java:

	getDocumentInfo(): Store the available Locales in the DocumentInfo.

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java:

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java:

	Add a ListBox to show the available locales. Add getLocaleSelector(),

	setLocaleList(), getSelectedLocale(), setSelectedLocale().

	* src/main/java/org/glom/web/client/event/LocaleChangeEvent.java

	* src/main/java/org/glom/web/client/event/LocaleChangeEventHandler.

	java: Add these classes.

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity.java:

	start(): Fill the locales ListBox. Handle its change event, firing a 

	LocaleChangeEvent.

	setPlace(): Show the selected locale as specified by the URL token.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java:

	* src/main/java/org/glom/web/client/activity/ListActivity.java:

	Handle LocaleChangeEvent, going to a new *Place with that locale.

	The placement of the ListBox is not pretty, and it currently uses the ID

	as a title, instead of "English", "Deutsch", "Espanola", etc, but it 

	is a start.

2012-01-17  Murray Cumming  <murrayc@murrayc.com>

	Search box: Show the search text from the URL token.

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java:

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java:

	Add setQuickFindText().

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	.java: setPlace(): Store the queryText if the place is a ListPlace, 

	and call TableSelectionView.setQuickFindText().

2012-01-17  Murray Cumming  <murrayc@murrayc.com>

	Allow use of translations via, for instance, &lang=de in the URL.

	* pom.xml: Use the unstable java-libglom 1.21 version.

	* src/main/java/org/glom/web/client/OnlineGlomService.java:

	* src/main/java/org/glom/web/client/OnlineGlomServiceAsync.java

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

	init(): Instead of calling TranslatableItem.set_current_locale() 

	(now removed), call ConfiguredDocument.setDefaultLocaleID().

	However, this is only for default locales, which are not needed to 

	change the locale in the URL.

	getDocumentInfo(), getListViewLayout(), getSortedListViewData(),

	getDetailsData(), getDetailsLayoutAndData(), getRelatedListData(),

	getSortedRelatedListData(): Add a localeID parameter, so we can get the 

	layout for a particular locale.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

	Add get/setDefaultLocaleID().

	getDocumentInfo(), getListViewData(), getRelatedListData(),

	getDetailsLayoutGroup(), getListViewLayoutGroup(),

	createLayoutItemPortalDTO(), convertToGWTGlomLayoutItemField(): Add a 

	localeID parameter, so we can get the layout for a particular locale.

	* src/main/java/org/glom/web/client/place/HasSelectableTablePlace.java:

	* src/main/java/org/glom/web/client/place/DetailsPlace.java:

	* src/main/java/org/glom/web/client/place/ListPlace.java:

	Parse and construct a lang parameter too.

	* src/main/java/org/glom/web/client/activity/DocumentSelectionActivity.java:

	start(): Pass the defaultLocaleID to addDocumentLink(). It is then

	passed to subsequent methods and constructors.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java:

	* src/main/java/org/glom/web/client/activity/ListActivity.java:

	Store the localeID from the *Place and pass it to other constructors

	and methods, such as OnlineGlomServiceAsync.getDetailsLayoutAndData().

	* src/main/java/org/glom/web/client/ui/DocumentSelectionView.java:

	* src/main/java/org/glom/web/client/ui/DocumentSelectionViewImpl.java:

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java

	* src/main/java/org/glom/web/client/ui/ListView.java:

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	Take localeID parameters and pass them to subsequent constructors and 

	methods, so that the layout is always retrieved for that locale.

	This is rather repetitive.

	Note that "" means the original (default) locale of the Glom document,

	which is usually English.	

2012-01-17  Murray Cumming  <murrayc@murrayc.com>

	Documents: Remove final keyword to fix startup configuration.

	* src/main/java/org/glom/web/shared/Documents.java: Remove the

	final keywords on the private member variables because that breaks

	the startup, apparently (there are warnings) because it stops them

	from being serialized. I added these in the previous commit.

2012-01-13  Murray Cumming  <murrayc@murrayc.com>

	Documents: Add some final keywords.

	* src/main/java/org/glom/web/shared/Documents.java: Eclipse suggested

	this.

2012-01-13  Murray Cumming  <murrayc@murrayc.com>

	OnlineGlomServiceImpl: Add to overview comments.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

	Note that this is where all the document are loaded. They are not 

	loaded freshly for each page.

2012-01-12  Murray Cumming  <murrayc@murrayc.com>

	Add a search box.

	* src/main/java/org/glom/web/client/ui/TableSelectionViewImpl.java:

	Add a TextBox for the text of a quick find.

	Add getQuickFindBox(), to get the widget, and getQuickFindText() to 

	get the text.

	setBackLink(): Add a String quickFind parameter.

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java

	(TableSelectionView): Add getQuickFindBox() and getQuickFindText()

	to the base interface, because that is how TableSelectionViewImpl is used.

	* src/main/webapp/style.css: Add style for the search box and its label.

	* src/main/java/org/glom/web/client/event/QuickFindChangeEvent.java:

	* src/main/java/org/glom/web/client/event/QuickFindChangeEventHandler.java:

	Add these files, based on the existing TableChangeEvent and 

	TableChangeEventHandlers.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java

	start(): Handle QuickFindChangeEvent, passing its quickFind text to

	a ListPlace() that the user should be taken to.

	* src/main/java/org/glom/web/client/activity/ListActivity.java

	start(): Handle it here too and adapt the TableChangeEvent handler to 

	pass the extra "" quickFind parameter to ListPlace.

	* src/main/java/org/glom/web/client/place/ListPlace.java: 

	Constructor: Take an extra String quickFind parameter and store it, 

	returning it from a new  getQuickFind() method.

	getToken(): Put the quickFind text in the URL token.

	getPlace(): Parse the quickFind text from the URL token.

	* src/main/java/org/glom/web/client/ui/DocumentSelectionViewImpl.ja

	va: addDocumentLink(): Pass an extra "" quickFind parameter to the 

	ListPlace constructor.

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	.java: start(): Add a Change handler for the TableSelectionView's

	TextBox (via its base HasChangeHandlers interface), firing the new 

	QuickFindChangeEvent.

	setPlace(): Adapt the call to TableSelectionView.setbackLink(), to 

	pass the extra "" quickFind parameter.

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	setCellTable(): Add a String quickFind parameter and pass it to 

	the ListViewTable() constructor.

	* src/main/java/org/glom/web/client/ui/ListView.java: Change 

	setCellTable() in the base interface, because that is how ListViewImpl

	is used.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

	Add a String quickFind member variable.

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java:

	Constructor: Add a String quickFind parameter, storing it in the

	base ListTable's member variable.

	onRangeChanged(): Pass quickFind to the 

	OnlineGlomServiceAsync.getSortedListViewData() and 

	OnlineGlomServiceAsync.getListViewData() methods.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

	getListViewData(), getSortedListViewData(): Add a String quickFind 

	parameter, passing it to ConfiguredDocument.getListViewData().

	* src/main/java/org/glom/web/client/OnlineGlomService.java:

	Change getListViewData(), getSortedListViewData() in the base interface,

	because that is how OnlineGlomServiceImpl is used, via this:

	* src/main/java/org/glom/web/client/OnlineGlomServiceAsync.java:

	Change getListViewData(), getSortedListViewData() here too.

	This class can apparently be used to asynchronously call methods on 

	OnlineGlomService, and GWT seems to implement that after recognizing 

	just the *Async name convention and the extra AsyncCallback parameters.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java

	getListViewData(): Add a String quickFind parameter, and pass it to 

	ListViewDBAccess.getData().

	* src/main/java/org/glom/web/server/database/ListDBAccess.java

	getListData(): Add a String quickFind parameter and pass it to 

	getSelectQuery().

	getSelectQuery(): Add a String quickFind parameter.

	* src/main/java/org/glom/web/server/database/ListViewDBAccess.java

	getSelectQuery(): Add a String quickFind parameter and use it with 

	Glom.get_find_where_clause_quick() to pass a where_clause to 

	Glom.build_sql_select_with_where_clause(), to actually filter the 

	list view results.

	getData(): Add a String quickFind parameter, passing it to getListData().

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.ja

	va: getData(): Pass an empty string to getListData() for the 

	quickFind parameter.

2012-01-12  Murray Cumming  <murrayc@murrayc.com>

	ListTable: Minor change.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java

	createCellTable(): Make this protected instead of public.

2012-01-12  Murray Cumming  <murrayc@murrayc.com>

	Many files: Use final for the parameters and use the @override attribute.

2012-01-22  Ben Konrath <ben@bagu.org>

	Add anchor links for single line text that starts with http, ftp and www.

	Bug #667269

2012-01-22  Ben Konrath <ben@bagu.org>

	Add ellipsis to single line text in details view.

	Bug #667269

2012-01-04  Murray Cumming  <murrayc@murrayc.com>

	Remove all javadoc author tags.

	Because they are awkward and meaningless when many people touch

	many files.

	See https://gitorious.org/online-glom/gwt-glom/commit/7628b732cb90cbc6d5635420a75568504e8b3655#comment_81164

2012-01-04  Murray Cumming  <murrayc@murrayc.com>

	Revert the COPYING.LESSER to COPYING rename.

	Apparently both should be there if it is LGPL.

2012-01-03  Murray Cumming  <murrayc@murrayc.com>

	*View: Remove unused imports.

	* src/main/java/org/glom/web/client/ui/DetailsView.java:

	* src/main/java/org/glom/web/client/ui/DocumentSelectionView.java

	* src/main/java/org/glom/web/client/ui/ListView.java:

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java:

	Remove unused imports, as suggested by Eclipse.

2012-01-02  Murray Cumming  <murrayc@murrayc.com>

	Move the *View::Presenter types, and some API into one base View.

	* src/main/java/org/glom/web/client/ui/DetailsView.java:

	* src/main/java/org/glom/web/client/ui/DocumentSelectionView.java:

	* src/main/java/org/glom/web/client/ui/ListView.java:

	* src/main/java/org/glom/web/client/ui/TableSelectionView.java: Move

	Presenter, setPresenter() and clear() into a shared base interface,

	to avoid the unnecessary duplicate Presenter types and to more clearly

	show how the *Views share the same structure, even if they are not 

	used polymorphically.

	* src/main/java/org/glom/web/client/ui/DetailsViewImpl.java:

	* src/main/java/org/glom/web/client/ui/DocumentSelectionViewImpl.ja

	va:

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java:

	* src/main/java/org/glom/web/client/activity/ListActivity.java:

	* src/main/java/org/glom/web/client/activity/DocumentSelectionActiv

	ity.java:

	* src/main/java/org/glom/web/client/activity/TableSelectionActivity

	.java: Adapt.

	Feel free to revert this if there is a good reason for the duplicate

	types.

2012-01-02  Murray Cumming  <murrayc@murrayc.com>

	OnlineGlom: Make clientFactory a (protected) member, and test it a bit.

	* src/main/java/org/glom/web/client/OnlineGlom.java: Make clientFactory

	a class member instead of a local variable in the method.

	This lets us use it to get the view instances, for use in tests.

	* src/test/java/org/glom/web/client/GwtTestOnlineGlom.java:

	beforeOnlineGlom(): Test some more details of the initial view.

	Again, this is not very useful.

	To really test gwt-glom we will need to start a local postgresql 

	instance with local data, like the Glom tests in C++.

2012-01-02  Murray Cumming  <murrayc@murrayc.com>

	pom.xml: Mention the LGPL license.

	* pom.xml: Add a licenses section.

	* COPYING.LESSER: Move this to COPYING, which

	previously contained the GPL. But gwt-glom is all LGPL.

2012-01-02  Murray Cumming  <murrayc@murrayc.com>

	Add project information to README and pom.xml.

	* README: Add a brief description and mention some mvn

	commands.

	* pom.xml: This extra information shows up in mvn site

	generated pages.

2011-01-02  Murray Cumming  <murrayc@murrayc.com>

	Use the latest java-libglom version.

	* pom.xml: Use java-libglom 1.19.2 instead of 1.19.1.

2012-01-01  Murray Cumming  <murrayc@murrayc.com>

	GwtTestOnlineGlom: Test a little more.

	* src/main/java/org/glom/web/client/OnlineGlom.java: Make the panels

	protected rather than private, as suggested by the gwt-test-utils

	maintianer here:

	http://stackoverflow.com/questions/7931724/gwt-testcase-simulating-clicking-a-button-on-my-page

	* src/test/java/org/glom/web/client/GwtTestOnlineGlom.java

	Test the initial visibility of the panels.

	However, this is not a very useful test.

	And I wonder how we should generally test using this idea for an

	activity/places app like ours where the real changes happen implicitly

	based on the history token/URL.

2012-01-01  Murray Cumming  <murrayc@murrayc.com>

	Slight modification to *Mapper comments.

	* src/main/java/org/glom/web/client/mvp/DataActivityMapper.java

	(DataActivityMapper)

	* src/main/java/org/glom/web/client/mvp/DocumentSelectionActivityMa

	pper.java

	* src/main/java/org/glom/web/client/mvp/TableSelectionActivityMappe

	r.java

	Remove comments mentioning GIN because they are just copied from 

	the example code and are apparently not helpful:

	http://groups.google.com/group/google-web-toolkit/msg/82f0098b20669a73

	Also change the mention of a class that is only in the example code.

2012-01-01  Murray Cumming  <murrayc@murrayc.com>

	GwtTestOnlineGlom test: Minor changes.

	* src/test/java/org/glom/web/client/GwtTestOnlineGlom.java:

	Avoid the long qualified class name and modify the comment 

	because it is now obvious to me that the mocked class is the only

	custom one created via GWT.create().

2012-01-01  Murray Cumming  <murrayc@murrayc.com>

	Tests: Added the beginnings of a test using gwt-test-utils.

	* pom.xml: Add dependencies on gwt-test-utils and easymock.

	* src/test/resources/META-INF/gwt-test-utils.properties: Add this file

	which tells gwt-test-utils what class will be tested.

	* src/test/java/org/glom/web/client/GwtTestOnlineGlom.java:

	Add a simple (but empty) test case. One class, used by the OnlineGlom

	class, is mocked so that it can be created. However, I am not sure 

	why only this class needs to be mocked.

	Note that mockito seems more popular, and clearer, than easymock,

	but I have not got that working yet. It might be a matter of the 

	mockito version.

	This test is run during mvn integration-test.

2011-12-31  Murray Cumming  <murrayc@murrayc.com>

	Tests: Use junit4-style syntax instead of junit3-style.

	* src/test/java/org/glom/web/client/place/DetailsPlaceTest.java:

	* src/test/java/org/glom/web/client/place/ListPlaceTest.java:

	* src/test/java/org/glom/web/shared/DataItemTest.java:

	Use the @Test annotation rather than relying on the test*() prefix.

	Also no longer implement TestCase, to avoid triggering support for 

	the junit3-way, which stops the annotations from working.

	Change the imports from import junit.framework.* to 

	import org.junit.*, which is apparently the new way.

2011-12-31  Murray Cumming  <murrayc@murrayc.com>

	Added a test for ListPlace token parsing and creation.

	* src/test/java/org/glom/web/client/place/ListPlaceTest.java:

	This is much the same as DetailsPlaceTest.

	I wonder how we could test the other parts of the *Place API.

2011-12-30  Murray Cumming  <murrayc@murrayc.com>

	DetailsPlace test: Also test getToken() and recreation via getPlace().

	* src/test/java/org/glom/web/client/place/DetailsPlaceTest.java:

	testGetPlaceParameters(): Get the tokens from the DetailsPlace and 

	recreate it, testing the recreated DetailsPlace for the same parameter

	values.

2011-12-30  Murray Cumming  <murrayc@murrayc.com>

	Use the surefire-report plugin.

	* pom.xml: This generates a HTML report about the tests in 

	target/site/surefire-report.html

	when you do mvn surefire-report:report. It seems to be popular/normal.

2011-12-30  Murray Cumming  <murrayc@murrayc.com>

	Added a test for DetailsPlace.

	* src/test/java/org/glom/web/client/place/DetailsPlaceTest.java:

	Test the getPlace() token parsing.

2011-12-30  Murray Cumming  <murrayc@murrayc.com>

	Added a first unit test.

	* pom.xml: Add a test goal, and a dependency on junit in that scope.

	* src/test/java/org/glom/web/shared/DataItemTest.java:

	This is a silly test but it is just to get things started. Note that

	maven/junit finds the test because it looks in src/test by default.

2011-12-22  Ben Konrath  <ben@bagu.org>

	Change charsetName to "UTF-8" when replacing line breaks.

	JavaScript requires the charsetName to be "UTF-8". CharsetName values

	that work in Java (such as "UTF8") will not work when compiled to

	JavaScript.

	This fixes a problem with multi-line details view fields that have hard

	line breaks. The "License Text" field on this page demonstrates the

	problem:

	http://onlineglom.openismus.com/OnlineGlom/#details:document=debian_repository_analyzer&table=licenses&value=197

	* src/main/java/org/glom/web/client/ui/details/DetailsCell.java:

2011-12-22  Ben Konrath  <ben@bagu.org>

	Fix another bug with related list navigation.

	I've tested all the navigation buttons in all of the related lists

	so things should be good now.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

2011-12-22  Ben Konrath  <ben@bagu.org>

	Fix a crasher when refreshing the list view with the default table.

	This crash will also happen when loading the list view with the default

	table from a link or bookmark.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java: Go

	to the main document selection page when the document id hasn't been

	set.

	* src/main/java/org/glom/web/client/activity/ListActivity.java: Go to

	the main document selection page when the document id hasn't been

	set.

	* src/main/java/org/glom/web/client/place/DetailsPlace.java: Use empty

	values for the details place when the document id hasn't been set.

	* src/main/java/org/glom/web/client/place/ListPlace.java: Use empty

	values for the list place when the document id hasn't been set.

2011-12-21  Ben Konrath  <ben@bagu.org>

	Protect against NPE when glom.document.locale is not in config.

	This patch protects against an NPE when glom.document.locale is not in

	the config file. This NPE will also happen if glom.document.locale is

	commented out.

	The patch also updates the error message to display the class name when

	the getMessage() returns null. This was happening when the NPE was

	thrown and I had "Configuration Error: null". If an NPE is encountered

	with this patch, "Configuration Error: NullPointerException " will be

	displayed.

	This commit closes this bug:

	https://bugzilla.gnome.org/show_bug.cgi?id=666669

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

2011-12-20  Murray Cumming  <murrayc@murrayc.com>

	Rename onlineglom.properties to onlineglom.properties.sample.

	* src/main/resources/onlineglom.properties: Rename to:

	* src/main/resources/onlineglom.properties.sample:

	* src/main/resources/README: And add this file explaining that people

	should rename it back when deploying.

2011-12-20  Murray Cumming  <murrayc@murrayc.com>

	Allow choosing the translation in the .properties file.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java

	init(): Read a glom.document.locale value from the configuration file 

	and call Glom's TransatableItem::set_current_locale() method.

	* src/main/resources/onlineglom.properties: Add a commented-out 

	example of this new setting.

	It would be better to add &lang=de_DE to the URL, but the current 

	libglom API does not allow us to do this easily. I am working on that.

2011-12-19  Murray Cumming  <murrayc@murrayc.com>

	Avoid a crash in parsing of token parameters.

	* src/main/java/org/glom/web/client/place/HasSelectableTablePlace.j

	ava: getTokenParams(): Do not crash if a parameter has a key but no 

	value, and ignore parameters with neither.

2011-12-17  Murray Cumming  <murrayc@murayc.com>

	History token building/handling: Improve use of token parameters.

	* src/main/java/org/glom/web/client/place/HasSelectableTablePlace.java 

	(HasSelectableTablePlace.Tokenizer): Add getTokenParams(String)

	and buildParamsToken(HashMap), for use by derived classes.

	Make the separator private because it is no longer be needed.

	* src/main/java/org/glom/web/client/place/DetailsPlace.java

	(DetailsPlace.Tokenizer.getToken): Use buildParamsToken()

	instead of manual string concatenation.

	(DetailsPlace.Tokenizer.getPlace): Use getTokenParams() instead 

	of hardcoded indices and awkward splitting code.

	* src/main/java/org/glom/web/client/place/ListPlace.java

	(ListPlace.Tokenizer.getToken): Use buildParamsToken()

	instead of manual string concatenation.

	(ListPlace.Tokenizer.getPlace): Use getTokenParams() instead 

	of hardcoded indices and awkward splitting code.

	This should fix bug #666420

2011-12-16  Murray Cumming  <murrayc@murrayc.com>

	Fix a Navgiation->Navigation typo in the code.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java

	Rename processNavgiation() to processNavigation().

2011-12-16  Murray Cumming  <murrayc@murrayc.com>

	Fix a seperator->separator typo in the code.

	* src/main/java/org/glom/web/client/place/DetailsPlace.java

	* src/main/java/org/glom/web/client/place/HasSelectableTablePlace.java

	* src/main/java/org/glom/web/client/place/ListPlace.java: Just a 

	misspelling.

2011-12-15  Ben Konrath <ben@bagu.org>

	Cleanup some comments.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

2011-12-14  Ben Konrath <ben@bagu.org>

	Replace \n with <br/> for multiline text in the details view.

	Vertical scrollbars are added when needed as well.

	* src/main/java/org/glom/web/client/ui/details/DetailsCell.java:

2011-12-14  Ben Konrath <ben@bagu.org>

	Specify the font for document selection links.

	* src/main/webapp/style.css:

2011-12-14  Ben Konrath <ben@bagu.org>

	Fix bouncy CellTable while paging.

	This doesn't currently work with related list tables in unselected

	Notebook tabs.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java

2011-12-14  Ben Konrath <ben@bagu.org>

	Revamp the appearance of the document selection page.

	* src/main/java/org/glom/web/client/ui/DocumentSelectionViewImpl.java:

	* src/main/java/org/glom/web/client/ui/DocumentSelectionViewImpl.ui.xml:

	* src/main/webapp/style.css:

2011-12-13  Ben Konrath <ben@bagu.org>

	Set navigation button column to the smallest size possible.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

2011-12-13  Ben Konrath <ben@bagu.org>

	Change OpenButton nomenclature to NavigationButton.

	Using NavigtionButton makes things more generic. Classes, methods and

	variables have been changed.

	This is a rename-only refactor.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java:

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	* src/main/java/org/glom/web/client/ui/cell/NavigationButtonCell.java:

	Renamed from OpenButtonCell.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java:

2011-12-12  Ben Konrath <ben@bagu.org>

	Remove unnecessary String argument in RelatedListTable and ListViewTable.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java:

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java:

2011-12-12  Ben Konrath <ben@bagu.org>

	Update variable names and comments.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java:

2011-12-12  Ben Konrath <ben@bagu.org>

	Properly initialize numNonEmptyRows variable to zero.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Add latest mockup with HTML tables.

	Features of this mockup:

	-> HTML table for flowtable

	-> HTML table for flowtable column

	-> Example of how related lists would look

	-> Not using text entries for data items

	The current version of Online Glom doesn't use HTML tables for the

	flowtable columns.

	This mockup has been sent to the glom-devel mailing list but it's good

	to have it here as well.

	* mockups/details-view-html-tables.html:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Remove unnecessary getPrimaryKeyField() method.

	getPrimaryKeyFieldForTable(String) has been renamed to

	getPrimaryKeyField(String).

	* src/main/java/org/glom/web/server/database/DBAccess.java:

	* src/main/java/org/glom/web/server/database/DetailsDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Add string representation of TypedDataItem value to conversion error message.

	* src/main/java/org/glom/web/server/Utils.java: Logging the error

	message was extracted into its own method to avoid duplication.

2011-12-05  Ben Konrath  <ben@bagu.org>

	Add type checking to navigation primary key value creation.

	Create navigation primary key only if the expected type from the Glom

	document matches the type returned by the SQL query.

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Rename a couple of variables in RelatedListNavigation.

	This is a rename-only refactor.

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Move getListLayoutGroup() into getListViewLayoutGroup().

	This removes getListLayoutGroup(). It was only being called by

	getListViewLayoutGroup().

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Remove check for LayoutItem_Portal in list table method.

	This check is no longer necessary because the method isn't being used

	to create the LayoutItemPortal DTO.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

2011-12-05  Ben Konrath  <ben@bagu.org>

	Properly support related list navigation.

	Navigation from the "Repository Analyzer -> Package Scans ->

	Dependencies" related table wasn't working because the primary key for

	related tables wasn't being set properly. This commit fixes the

	problem.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java: Don't use

	getListLayoutGroup() to create the LayoutItemPortal DTO. This method

	doesn't set the primary key properly for related list tables.

	* src/main/java/org/glom/web/server/database/DBAccess.java: Add table

	name parameter to getPrimaryKeyLayoutItemField(). This makes the method

	useful for getting the primary key for list view tables and for related

	list tables.

	* src/main/java/org/glom/web/server/database/ListDBAccess.java:

	* src/main/java/org/glom/web/server/database/ListViewDBAccess.java:

	Move code to set the primary key for the table from the abstract

	ListDBAccess class to ListViewDBAccess as it's only correct for list

	view tables.

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	Properly add primary key to related list tables.

2011-12-02  Ben Konrath  <ben@bagu.org>

	Properly set the horizontal alignment of fields.

	This fix is for both the list tables and the details view.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java: Use

	LayoutItem_WithFormatting.get_formatting_used_horizontal_alignment(boolean)

	to set the horizontal alignment of fields.

2011-12-02  Ben Konrath  <ben@bagu.org>

	Display currency codes in the details view.

	* src/main/java/org/glom/web/client/ui/details/DetailsCell.java:

2011-12-02  Ben Konrath  <ben@bagu.org>

	Avoid duplicate JNI call.

	JNI is not as efficient as pure Java and this is an easy (and small)

	optimization.

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	Use previously retrieved value for whereClauseToTableName instead of

	getting it again.

2011-12-02  Ben Konrath  <ben@bagu.org>

	Rename a couple of variables in RelatedListNavigation.

	This is a rename-only refactor.

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

2011-12-02  Ben Konrath  <ben@bagu.org>

	Indicate clearly that a mismatched primary key type is a bug.

	* src/main/java/org/glom/web/server/Utils.java: Change log level from

	warning to error. Add 'This is a bug.' to message.

2011-12-02  Ben Konrath  <ben@bagu.org>

	Update / fix some comments.

	* src/main/java/org/glom/web/client/OnlineGlomService.java: Remove old

	comments.

	* src/main/java/org/glom/web/client/ui/details/DetailsCell.java: Fix

	comment.

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	Fix comments. Add some TODOs.

2011-12-02  Ben Konrath  <ben@bagu.org>

	Enable navigation to details view with string primary key from related list.

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

	Create a text primary key value when return type of result is

	java.sql.Types.VARCHAR.

2011-12-02  Ben Konrath  <ben@bagu.org>

	Use checkboxes for booleans in the details view.

	* src/main/java/org/glom/web/client/ui/details/DetailsCell.java:

2011-12-01  Ben Konrath  <ben@bagu.org>

	Improve performance of related list height calculation.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	Put code to calculate the expected height in a static initializer so

	that that it's only called once.

2011-12-01  Ben Konrath  <ben@bagu.org>

	Show related list tables in notebooks (again).

	Calculate the height of the related list tables so the Notebook can be

	set the correct height. The height of the related list table is also needed by

	FlowTable to be able decide how to create the layout.

	* src/main/java/org/glom/web/client/ui/details/Portal.java: Calculate

	and set the Portal height based on the height of the related list

	table and the Portal container.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	Add method to calculate the height of the related list tables.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

	* src/main/webapp/style.css: Add css class for Pager. This is needed to

	calculate the height of the Pager widget.

2011-12-01  Ben Konrath  <ben@bagu.org>

	Use CellTable API for table property instead of setting style on Element.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

2011-12-01  Ben Konrath  <ben@bagu.org>

	Make ListViewTable and RelatedListTable a consistent height.

	The tables are now a consistent height regardless of the contents of

	the table. A hidden button is added to empty rows to ensure that the

	height of these rows will match the height of rows with data.

	A navigation button column is now added to every table. The width of

	the navigation column is set to 0px when a RelatedListTable shouldn't

	have navigation buttons. This maintains the a consistent row height in

	tables that don't show the navigation buttons.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java: Hide

	navigation column when not needed.

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java: Move method

	arguments for navigation button to constructor of ListViewTable.

	* src/main/java/org/glom/web/client/ui/cell/OpenButtonCell.java: Render

	hidden button for empty data rows.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java: Add method

	arguments for navigation button to constructor.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java: Always

	create navigation buttons. Add hideNavigationButtons() method.

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java: Add method

	arguments for navigation button to constructor.

2011-12-01  Ben Konrath  <ben@bagu.org>

	Use 'visibility: hidden' in Utils.getWidgetHeight().

	This is better choice because hidden elements are invisible, don't

	respond to events and are not part of the tab order. They will,

	however, take up space which is required to be able to calculate the

	height of the widget.

	* src/main/java/org/glom/web/client/Utils.java:

2011-12-01  Ben Konrath  <ben@bagu.org>

	Use Utils.getWidgetHeight() in FlowTable.

	* src/main/java/org/glom/web/client/Utils.java: Remove TODO item about

	this.

	* src/main/java/org/glom/web/client/ui/details/FlowTable.java:

2011-12-01  Ben Konrath  <ben@bagu.org>

	Put the details css class name on the correct table column.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

2011-11-30  Ben Konrath  <ben@bagu.org>

	Update for java-libglom API change.

	The getters and setters on FieldFormatting and NumericFormat were

	changed to remove the 'M'.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

2011-11-29  Ben Konrath  <ben@bagu.org>

	Only allow RelatedListTables in Portals.

	* src/main/java/org/glom/web/client/ui/details/Portal.java:

2011-11-29  Ben Konrath  <ben@bagu.org>

	Only create a contents panel for Portals when title is being set.

	* src/main/java/org/glom/web/client/ui/details/Portal.java:

2011-11-29  Ben Konrath  <ben@bagu.org>

	Set TabLayoutPanel height based on calculated height its widgets.

	This is a potential fix for this bug:

	https://bugzilla.gnome.org/show_bug.cgi?id=665133

	* src/main/java/org/glom/web/client/ui/details/Notebook.java:

2011-11-29  Ben Konrath  <ben@bagu.org>

	Align details field labels and data with the Open buttons.

	* src/main/webapp/style.css:

2011-11-29  Ben Konrath  <ben@bagu.org>

	Remove unnecessary <div> in the Notebook widget.

	* src/main/java/org/glom/web/client/ui/details/Group.java: Remove

	method to get container FlowPanel (<div>).

	* src/main/java/org/glom/web/client/ui/details/Notebook.java: Run the

	initWidget() method directly on the TabLayoutPanel widget instead of

	Group's container widget.

2011-11-29  Ben Konrath  <ben@bagu.org>

	Don't add group titles for Portals in Notebooks.

	This reverts the previous patch and fixes a bug I introduced with

	commit b1753fd27bd2c4ea189c4c353e0ece92dcc66c2c .

	* src/main/java/org/glom/web/client/ui/details/Group.java:

	* src/main/java/org/glom/web/client/ui/details/Portal.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Remove unused boolean argument in Portal constructor.

	Just a code cleanup.

	* src/main/java/org/glom/web/client/ui/details/Group.java:

	* src/main/java/org/glom/web/client/ui/details/Portal.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Remove hack for glom 1.18 style glom files.

	* src/main/java/org/glom/web/client/ui/details/Group.java:

	* src/main/java/org/glom/web/client/ui/details/Notebook.java:

	* src/main/java/org/glom/web/client/ui/details/Portal.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Use Gda Value version of primary key to log result too large error.

	* src/main/java/org/glom/web/server/database/DetailsDBAccess.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Don't use TypedDataItem.getText() for Unknown types from the URL.

	* src/main/java/org/glom/web/client/place/DetailsPlace.java:

	* src/main/java/org/glom/web/server/Utils.java: Use getUnknown()

	instead of getText().

	* src/main/java/org/glom/web/shared/TypedDataItem.java: Add unknown

	String field and getUnknown() method.

2011-11-28  Ben Konrath  <ben@bagu.org>

	Log an error message when the java-libglom .so is not present.

	The error message was being set in the exception but not logged.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Ignore LayoutItem_CalendarPortals.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java: Don't

	create the LayoutItemPortal DTO for LayoutItem_CanendarPortals.

2011-11-28  Ben Konrath  <ben@bagu.org>

	Extract method for creating the LayoutItemPortal DTO.

	Just breaking the code up into smaller chunks.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Add TypedDataItem.

	This should have been added with the refactor. Oops!

	* src/main/java/org/glom/web/shared/TypedDataItem.java:

2011-11-28  Ben Konrath  <ben@bagu.org>

	Create primary key value from URL string using type from Glom document.

	See this bug, comments 19 - 25:

	https://bugzilla.gnome.org/show_bug.cgi?id=662376#c19

	* src/main/java/org/glom/web/client/place/DetailsPlace.java: Don't

	create a TypeDataItem for the primary key here when loading from a

	URL. Show the same string for the primary key value as was received

	from the URL string (when loading from a URL).

	* src/main/java/org/glom/web/server/Utils.java: Update method for

	creating the Gda Value from the TypeDataItem to properly deal with

	creating a Gda Value based on the Glom document type for the primary

	key value string when loading from a URL.

	* src/main/java/org/glom/web/server/database/DetailsDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

	Update for changed method name.

2011-11-27  Ben Konrath  <ben@bagu.org>

	Rename PrimaryKeyItem to TypedDataItem.

	The name PrimaryKeyItem suggests what the class should be used for.

	TypedDataItem is a neutral name that describes the class better.

	This is a rename-only refactor.

	* src/main/java/org/glom/web/client/OnlineGlomService.java:

	* src/main/java/org/glom/web/client/OnlineGlomServiceAsync.java:

	* src/main/java/org/glom/web/client/Utils.java:

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java:

	* src/main/java/org/glom/web/client/place/DetailsPlace.java:

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

	* src/main/java/org/glom/web/server/Utils.java:

	* src/main/java/org/glom/web/server/database/DetailsDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

	* src/main/java/org/glom/web/shared/NavigationRecord.java:

2011-11-25  Ben Konrath  <ben@bagu.org>

	Improve Gda Value conversion from PrimaryKeyItem.

	The value from the PrimaryKeyItem is only used if its type match the

	type from the glom document.

	* src/main/java/org/glom/web/server/Utils.java:

2011-11-25  Ben Konrath  <ben@bagu.org>

	Manually check if the java-liblgom .so is visible to the JVM.

	It seems that Tomcat has problems when a static initializer throws an

	exception. This check is done before the first method call into

	java-libglom so that execution doesn't continue if the .so is not

	found.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

2011-11-25  Ben Konrath  <ben@bagu.org>

	Improve browser configuration error messages.

	This fixes:

	https://bugzilla.gnome.org/show_bug.cgi?id=662792

	* src/main/java/org/glom/web/client/OnlineGlomService.java:

	* src/main/java/org/glom/web/client/OnlineGlomServiceAsync.java: Add

	getConfigurationErrorMessage() method.

	* src/main/java/org/glom/web/client/activity/DocumentSelectionActivity.java:

	Get and display a specific configuration error message when no Glom

	documents are found.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java:

	Implement getConfigurationErrorMessage() method. Surround configuration

	code in the init() method with a try/catch block. This allows the

	errors to be caught while keeping the servlet available to retrieve the

	configuration error message.

2011-11-25  Ben Konrath  <ben@bagu.org>

	Don't use Strings to hold primary key values.

	The primary key values are now held in a new data object

	(PrimaryKeyItem) that holds type information and the primary key value

	using the correct type.

	* src/main/java/org/glom/web/client/OnlineGlomService.java:

	* src/main/java/org/glom/web/client/OnlineGlomServiceAsync.java:

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java: Use

	PrimaryKeyItem instead of String to hold the primary key value.

	* src/main/java/org/glom/web/client/Utils.java: Remove

	getKeyValueStringForQuery(). Add getPrimaryKeyItem() which creates a

	PrimaryKeyItem based on the GlomFieldType and the DataItem.

	* src/main/java/org/glom/web/client/activity/DetailsActivity.java: Use

	PrimaryKeyItem instead of String to hold the primary key value. Load

	document selection page when the documentID has not been set correctly.

	* src/main/java/org/glom/web/client/place/DetailsPlace.java: Re-work

	DetailsPlace -> URL and URL -> DetailsPlace conversion with

	PrimaryKeyItem.

	* src/main/java/org/glom/web/client/place/DocumentSelectionPlace.java:

	Return empty string for URL instead of "null".

	* src/main/java/org/glom/web/client/ui/ListViewImpl.java:

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	* src/main/java/org/glom/web/client/ui/list/ListTable.java:

	* src/main/java/org/glom/web/server/ConfiguredDocument.java: Use

	PrimaryKeyItem instead of String to hold primary key values.

	* src/main/java/org/glom/web/server/Utils.java: New method to convert a

	PrimaryKeyValue to a Gda Value.

	* src/main/java/org/glom/web/server/database/DetailsDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

	Replace temporary database access code that uses the PrimaryKeyValue to

	Gda Value conversion.

	* src/main/java/org/glom/web/shared/DataItem.java: Add comment.

	* src/main/java/org/glom/web/shared/NavigationRecord.java: Use

	PrimaryKeyItem instead of String.

	* src/main/java/org/glom/web/shared/PrimaryKeyItem.java: New class to

	hold primary key values.

2011-11-24  Ben Konrath  <ben@bagu.org>

	Use newly added java-libglom API to create queries.

	This isn't finished. I still need to stop using Strings for primary key

	values in the client code.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java: Tell

	libglom to use fake connections so that retrieving the query string will

	work.

	* src/main/java/org/glom/web/server/database/DetailsDBAccess.java:

	* src/main/java/org/glom/web/server/database/ListViewDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListDBAccess.java:

	* src/main/java/org/glom/web/server/database/RelatedListNavigation.java:

	Use the newly added libglom sql methods and classes to create the

	query. Add temporary hack to convert primary value strings to Gda

	Value.

2011-11-23  Ben Konrath  <ben@bagu.org>

	Don't explicitly set the height of Portals.

	See comments 6 - 10 of this bug for details:

	https://bugzilla.gnome.org/show_bug.cgi?id=662930#c6

	* src/main/java/org/glom/web/client/ui/details/Portal.java:

2011-11-23  Ben Konrath  <ben@bagu.org>

	Use an HTML table instead of CSS for the FlowTable layout.

	* src/main/java/org/glom/web/client/ui/details/FlowTable.java: Use

	GWT's FlexTable to implement the FlowTable.

	* src/main/webapp/style.css: Adjust CSS for the change to FlexTable.

2011-11-18  Ben Konrath  <ben@bagu.org>

	Add boolean example to HTML table mockup.

	* mockups/details-view-html-tables-text-entries.html:

2011-11-17  Ben Konrath  <ben@bagu.org>

	Ensure the pager buttons are always visible for related lists.

	To accomplish this, I've turned off text wrapping in the list view and

	related list tables for both the header and data text. The related list

	table now has a fixed layout so the it doesn't overflow its container.

	This is required to ensure that the cell text is clipped when it

	overflows the cell and an ellipsis is added to the right side of the

	cell when text is clipped.

	A fixed table layout for the related list table in the details view

	seems what we want for the details view anyway, so the side-effect is

	desirable.

	The ellipsis will only be displayed in Firefox >= 7.

	This fixes bug:

	https://bugzilla.gnome.org/show_bug.cgi?id=662930

	* src/main/java/org/glom/web/client/ui/cell/NumericCell.java:

	* src/main/java/org/glom/web/client/ui/cell/TextCell.java: Add

	'overflow: hidden; text-overflow: ellipsis;' CSS properties to the table

	cell text.

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	Set the 'table-layout: fixed' CSS property to the related list table.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java: Set the

	'white-space: nowrap;' CSS property on both the list view and the

	related list tables.

2011-11-16  Ben Konrath  <ben@bagu.org>

	Rework the fix for empty notebook tab labels.

	Setting the empty group titles with its name caused problems for the

	details	layout. Instead of using libglom's

	LayoutItem.get_title_or_name(), the LayoutItem name is explicitly sent

	to the client when the title is	empty. This allows the Notebook to use

	the name when the title is empty without affecting anything else.

	* src/main/java/org/glom/web/client/ui/details/Notebook.java:

	* src/main/java/org/glom/web/server/ConfiguredDocument.java:

2011-11-16  Ben Konrath  <ben@bagu.org>

	Set group titles with name when title is empty.

	This fixes a problem with an empty notebook tab label in the Lesson

	Planner document. The forth tab in the notebook should be "Internet":

	http://bagu.org:8080/OnlineGlom/#details:document=lesson-planner&table=teachers&value=0

	* src/main/java/org/glom/web/server/ConfiguredDocument.java: Use

	libglom's LayoutItem.get_title_or_name() to fill in the LayoutGroup

	DTO title.

2011-11-16  Ben Konrath  <ben@bagu.org>

	Remove whitespace from the configured username properties.

	This assumes that usernames won't have whitespace at the beginning

	or end. But I think this is a reasonable assumption.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java: Use

	String.trim() to remove the whitespace from the username properties.

2011-11-15  Ben Konrath  <ben@bagu.org>

	Add details view mockup with HTML tables and text entries.

	This is from the attachment on this bug:

	https://bugzilla.gnome.org/show_bug.cgi?id=663109

	* mockups/details-view-html-tables-text-entries.html:

2011-11-15  Ben Konrath  <ben@bagu.org>

	Add space between the columns of the flow table.

	This fixes:

	https://bugzilla.gnome.org/show_bug.cgi?id=662918

	* src/main/java/org/glom/web/client/ui/details/FlowTable.java: Add a 1%

	space between columns in the flow table.

2011-11-15  Ben Konrath  <ben@bagu.org>

	Add backup files to the .gitignore.

	* .gitignore: Ignore files that end with ~.

2011-11-09  Ben Konrath  <ben@bagu.org>

	Use latest release of gwt-log.

	Gwt-log releases are now being submitted to the maven central

	repository so manual installation of the jar is no longer required.

	* pom.xml: Update version and groupId of gwt-log dependency.

2011-10-31  Ben Konrath  <ben@bagu.org>

	Don't use GWT numeric formatting to override the glom currency formatting.

	Currencies are now displayed like they are in Glom. See this bug:

	https://bugzilla.gnome.org/show_bug.cgi?id=646216

	* src/main/java/org/glom/web/client/Utils.java: Remove GWT currency

	formatting.

	* src/main/java/org/glom/web/client/ui/cell/NumericCell.java: Add

	currency code to constructor and set it when the cell is rendered.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java: Add the

	currency code to the constructor of the NumericCell.

2011-10-27  Ben Konrath  <ben@bagu.org>

	Require the latest release of java-libglom (1.17.4).

	* pom.xml:

2011-10-26  Ben Konrath  <ben@bagu.org>

	Add style to Notebook that matches current theme.

	It's not the best style in the world but it's better than the default.

	* src/main/java/org/glom/web/client/ui/details/Notebook.java: Add some

	padding at the bottom of the child widgets.

	* src/main/webapp/style.css: Add style for the Notebook.

2011-10-26  Ben Konrath  <ben@bagu.org>

	Move servlet initialization code to overridden init method.

	This is half of the solution to getting proper error messages

	displayed when configuration errors occur. Here's the relevant bug:

	https://bugzilla.gnome.org/show_bug.cgi?id=662792

	The rest of the solution involves surrounding the init method with a

	try/catch block and setting a global variable with the error /

	exception. A new async method should be created to retrieve and display

	the error message / exception.

	* src/main/java/org/glom/web/server/OnlineGlomServiceImpl.java: Move

	code from constructor to init method adding exceptions as needed.

2011-10-26  Ben Konrath  <ben@bagu.org>

	Add script to monitor and restart tomcat if required.

	* utils/check-and-recover-tomcat.py: New file.

2011-10-26  Ben Konrath  <ben@bagu.org>

	Display the correct number of data items in the pager.

	This commit fixes:

	https://bugzilla.gnome.org/show_bug.cgi?id=661441

	* src/main/java/org/glom/web/client/ui/list/ListViewTable.java:

	* src/main/java/org/glom/web/client/ui/details/RelatedListTable.java:

	The implementation is the same for both tables: Keep track of the

	number of non-empty rows and fire and RowCountChangeEvent after the data has

	been updated.

	* src/main/java/org/glom/web/client/ui/list/ListTable.java: Add a

	custom Pager class that subclasses SimplePager to handle displaying

	the correct number when empty rows have been added.

2011-10-26  Ben Konrath  <ben@bagu.org>

	Correct error in previous commit.

	* src/main/java/org/glom/web/client/activity/ListActivity.java: Remove

	eventBus parameter from listView.setCellTable().

2011-10-26  Ben Konrath  <ben@bagu.org>

	Fix error in TODO comment.

	* src/main/java/org/glom/web/client/activity/ListActivity.java:

2011-10-24  Ben Konrath  <ben@bagu.org>

	Create Notebook widgets to the details view.

	This isn't finished just yet - I still need to create a reasonable

	style to match the current theme.

	* src/main/java/org/glom/web/client/Utils.java: Add method for

	calculating the height of a widget. This is used in the Notebook class.

	* src/main/java/org/glom/web/client/ui/DetailsViewImpl.java: Update for

	new constructor method in Group.

	* src/main/java/org/glom/web/client/ui/details/Group.java: Add new

	method for creating child widget that can be used by subclasses

	like Notebook. New constructor that allows disabling the group

	titles - Notebooks don't set a group title for their child groups.

	* src/main/java/org/glom/web/client/ui/details/Notebook.java: New class

	to make Notebooks using GWT's TabLayoutPanel.

	* src/main/java/org/glom/web/client/ui/details/Portal.java: New

	constructor that allows disabling the group titles.

	* src/main/java/org/glom/web/server/ConfiguredDocument.java: Create the

	LayoutItemNotebook DTO.

	* src/main/java/org/glom/web/shared/layout/LayoutItemNotebook.java: New

	DTO for Notebooks. It's just an empty class for now but we might need

	it to transfer some specific information in the future.