Update zimply.py

[aeryskyB]

Addresses the issue regarding disabled fts4 in recent sqlite3 versions.
Updates (minimal) the syntax to be consistent with fts5.

• Fixes Error while index creation during first time server start-up #36

[kimbauters]

This specific fix would only work if you have FTS5, which isn't a guarantee for older installs. This is handled more carefully in the version2 branch by testing/checking for the highest supported FTS version and adapting the app accordingly:

ZIMply/zimply/zim_core.py

Lines 1429 to 1453 in 237364d

	def _highest_fts_level():
	# test FTS support in SQLite3; return True, False, or None when only available when loading extension
	def verify_fts_level(level):
	# try to create an FTS table using an in-memory DB, or try to explicitly load the extension
	tmp_db = sqlite3.connect(":memory:")
	try:
	tmp_db.execute("CREATE VIRTUAL TABLE capability USING fts" + str(level) + "(title);")
	except sqlite3.Error:
	try:
	tmp_db.enable_load_extension(True)
	tmp_db.load_extension("fts" + str(level))
	except sqlite3.Error:
	return False
	return None
	finally:
	tmp_db.close()
	return True
	if verify_fts_level(5) is True:
	return 5
	if verify_fts_level(4) is True:
	return 4
	if verify_fts_level(3) is True:
	return 3
	return None

.

Either the version2 branch needs to be (finally) mainlined – which still requires work to update the server for Python2 support, e.g. using https://github.com/chai-project/chai-backend-api/blob/f4df68ef0ae9e859e891949ec012faa220029f1e/chai_api/main.py#L21-L28 as well as an update to the html interface. Alternatively, this more advanced handling of FTS needs to be backported to the older main branch.

[aeryskyB]

@kimbauters,
I see your concern.
However, if any new user (who I suppose will use python3) goes for pip install zimply, I think, will face the same issue.

Best!