GitLab

* glom/libglom/data_structure/fieldtypes.[h|cc]: Fallback to a hard-coded map of
database types to GType types, for the (so far unexplained) cases where
CONNECTION_META_TYPES gives us no rows.

* glom/mode_data/db_adddel/db_adddel.[h|cc]:
Add signal_record_selection_changed and emit it when the TreeView's selection
changes:
* glom/mode_data/db_adddel/db_adddel_withbuttons.cc: Make sure that we call
the base class.
* glom/mode_data/box_data_manyrecords.[h|cc]:
Added signal_record_selection_changed.
* glom/mode_data/box_data_list.[h|cc]: Handle the AddDel_DB's
signal_record_selection_changed, emitting our own
signal_record_selection_changed.
* glom/mode_data/notebook_data.[h|cc]: Handle the Box_Data_List's
signal_record_selection_changed, emitting our own
signal_record_selection_changed.
I still do not like the repetition that is required here, and elsewhere,
when a child widget needs to signal all the way up an ownership hierarchy.

* glom/appwindow.[h|cc]: Add enable_menu_print_layouts_details().
* glom/frame_glom.[h|cc]: Handle the Notebook_Data's
signal_record_selection_changed(), calling the AppWindow's
enable_menu_print_layouts_details().

* glom/appwindow.cc: fill_menu_print_layouts(): Show the print layouts
even if the list view is visible.
* glom/mode_data/notebook_data.[h|cc]:
get_found_set_details(): Replace with get_found_set_selected()
which checks the selected record in the list view, if the list
view is visible, instead of just the details view.
* glom/frame_glom.[h|cc]: Remove get_viewing_details().
do_print_layout(): Use get_found_set_selected() so we can print the
print layout for the selected record.

This avoids some confusion caused by the print layouts not being available
when viewing the list instead of the details.
Bug #670462 (alien)

* glom/libglom/privs.[h|cc]: get_database_users(): Add documentation
about the case that the result is empty.
* glom/mode_design/users/dialog_users_list.cc: on_button_user_add():
If the users list is empty then warn that something is probably wrong
with the permissions. I would like to warn much earlier if the user is
not a superuser, but I do not know how.
Bug #669178 (alien)

Rename Application to AppWindow and rename the Bakery::App* classes
appropriately too. This is a preparation for using Gtk::Application.

* glom/print_layout/canvas_print_layout.cc: fill_with_data(): Do nothing,
and hint on stdout, if there are no records yet. This avoids several
real warnings later.

* glom/print_layout/canvas_layout_item.[h|cc]: Remove the constructor and
create() that take a LayoutItem, because we should put the CanvasItem in a
canvas (even indirectly) before creating the child item, to avoid this
goocanvas bug: https://bugzilla.gnome.org/show_bug.cgi?id=657592#c16
* glom/mode_design/print_layouts/window_print_layout_edit.[h|cc]:
* glom/print_layout/canvas_print_layout.[h|cc]:
Add and use create_canvas_layout_item_and_add() methods which put the new
canvas item in the canvas before creating its child items based on the
LayoutItem.

This fixes bug #668901 (alien)

* glom/mode_data/box_data.cc: :execute_button_script(): Move the
warning UI into
* glom/utils_ui.[h|cc]: a new script_check_for_pygtk2_with_warning()
method.
* glom/mode_design/fields/dialog_fieldcalculation.cc: on_button_test():
* glom/mode_design/layout/layout_item_dialogs/dialog_buttonscript.cc:
on_button_test_script(): Use the new function here.

* glom/libglom/utils.[h|cc]: Add script_check_for_pygtk().
* Makefile_tests.am:
* tests/test_script_check_for_problems.cc: Add a test for the check
function.
* glom/mode_data/box_data.cc: execute_button_script(): Show a
warning dialog, and do not run the script, if it seems to use pygtk.
This should help with but #669196 (alien) and ##661766 (Andre Klapper) .

* glom/libglom/report_builder.[h|cc]: Return bool
from the private methods and check those results.

* glom/libglom/report_builder.cc: report_build_summary():
Remove any sort clause (ORDER BY) from the FoundSet because that makes
no sense for a single row with summary fields. It looks like recent versions
of PostgreSQL have become more strict about this, producing this error, for
instance:
column invoices.invoice_id must appear in the GROUP BY clause or be used in an aggregate function
* Makefile_tests.am:
* tests/test_selfhosting_new_then_report_summary.cc: Add a test for this.
This fixes bug #669281 (alien)

	* glom/libglom/db_utils.[h|cc]: add_group(): Take a bool superuser
	parameter.
	add_groups_from_document(): Use add_group() instead of executing
	the ADD GROUP query directly, to make sure that we also start
	with some sane table privileges defaults, in case nothing else
	sets the privielges.
	* glom/libglom/document/document.cc: load_after(): Fix a typo
	so that the group privileges are really loaded (and then
	saved again instead of being lost).
	* examples/example_film_manager.glom:
	* examples/example_smallbusiness.glom: Save these with
	useful group privileges, so the groups will be allowed to
	see tables.
	* tests/test_document_load.cc: Test the loading of the group
	privileges.
	Previously the examples were created with groups that could not
	even view the tables, which meant that they could not even
	find out about their existence.
	This deals with the problem found here:
	https://bugzilla.gnome.org/show_bug.cgi?id=669043#c2
	But I would like to show the existence (and structure) of tables
	even if their data cannot be viewed, so there is more work to do.

	* tests/test_selfhosting_new_from_example_operator.cc: Check
	that the group exists before adding a user to it.

	* glom/libglom/db_utils.cc: get_table_names_from_database():
	Slightly improved error checking.
	* glom/libglom/privs.cc: set_table_privileges():
	Warn if a GRANT fails.

* glom/frame_glom.cc: Add a stdout message saying the the
default user has been disabled, because the existing message
about the failure to remove the user looks at first like
a security problem.
I still do not know why removal fails, even though we do
change the ownership of the database.

* glom/base_db.[h|cc]: Move remove_user() and
remove_user_from_group() to
* glom/libglom/db_utils.[h|cc].
* glom/frame_glom.cc:
* glom/mode_design/users/dialog_users_list.cc: Adapted.
* tests/test_selfhosting_new_empty_then_users.cc: Test these
functions.