[IMP] remove redundant SQL query and grep for missing addons
The SQL query already filters on module_origin.state = 'installed', so the second query to get installed addons and the grep intersection were completely redundant. Before: 2 SQL queries + grep + 3 temp files After: 1 SQL query + variable This simplifies the code and reduces database round-trips.
This commit is contained in:
@@ -42,7 +42,7 @@ echo "Base neutralized..."
|
|||||||
## List add-ons not in final version ##
|
## List add-ons not in final version ##
|
||||||
#######################################
|
#######################################
|
||||||
|
|
||||||
SQL_404_ADDONS_LIST=$(cat <<EOF
|
SQL_MISSING_ADDONS=$(cat <<EOF
|
||||||
SELECT module_origin.name
|
SELECT module_origin.name
|
||||||
FROM ir_module_module module_origin
|
FROM ir_module_module module_origin
|
||||||
LEFT JOIN (
|
LEFT JOIN (
|
||||||
@@ -56,20 +56,14 @@ WHERE (module_dest.name IS NULL)
|
|||||||
ORDER BY module_origin.name;
|
ORDER BY module_origin.name;
|
||||||
EOF
|
EOF
|
||||||
)
|
)
|
||||||
echo "Retrieve 404 addons... "
|
echo "Retrieve missing addons..."
|
||||||
echo "SQL REQUEST = $SQL_404_ADDONS_LIST"
|
missing_addons=$(query_postgres_container "$SQL_MISSING_ADDONS" "$DB_NAME")
|
||||||
query_postgres_container "$SQL_404_ADDONS_LIST" "$DB_NAME" > "${TMPDIR}/404_addons"
|
|
||||||
|
|
||||||
INSTALLED_ADDONS="SELECT name FROM ir_module_module WHERE state='installed';"
|
|
||||||
query_postgres_container "$INSTALLED_ADDONS" "$DB_NAME" > "${TMPDIR}/installed_addons"
|
|
||||||
|
|
||||||
grep -Fx -f "${TMPDIR}/404_addons" "${TMPDIR}/installed_addons" > "${TMPDIR}/final_404_addons" || true
|
|
||||||
|
|
||||||
echo "
|
echo "
|
||||||
==== ADD-ONS CHECK ====
|
==== ADD-ONS CHECK ====
|
||||||
Installed add-ons not available in final Odoo version:
|
Installed add-ons not available in final Odoo version:
|
||||||
"
|
"
|
||||||
cat "${TMPDIR}/final_404_addons"
|
echo "$missing_addons"
|
||||||
|
|
||||||
|
|
||||||
echo "
|
echo "
|
||||||
|
|||||||
Reference in New Issue
Block a user