From: Sarah Hoffmann <lonvia@denofr.de>
Date: Sat, 22 Sep 2018 08:47:01 +0000 (+0200)
Subject: Merge pull request #1187 from mtmail/faq-about-pear-db-warning
X-Git-Tag: v3.3.0~89
X-Git-Url: https://git.openstreetmap.org/nominatim.git/commitdiff_plain/1f887a6ca0aba36261c443a9077eb28725838f7c?hp=eba6e46c74abfc828f385fa883332f85a4a429f6

Merge pull request #1187 from mtmail/faq-about-pear-db-warning

Installation FAQ entry about a PHP warning that started with PHP7.2
---

diff --git a/docs/admin/Faq.md b/docs/admin/Faq.md
index 46da7e21..a0e1cafe 100644
--- a/docs/admin/Faq.md
+++ b/docs/admin/Faq.md
@@ -93,6 +93,14 @@ However, you can solve this the quick and dirty way by commenting out that line
     sudo systemctl restart httpd
 
 
+### "must be an array or an object that implements Countable" warning in /usr/share/pear/DB.php
+
+As reported starting PHP 7.2. This external DB library is no longer maintained and will be replaced in future Nominatim versions. In the meantime you'd have to manually change the line near 774 from
+`if (!count($dsn)) {` to  `if (!$dsn && !count($dsn))`. [More details](https://github.com/openstreetmap/Nominatim/issues/1184)
+
+
+
+
 ### Website reports "DB Error: insufficient permissions"
 
 The user the webserver, e.g. Apache, runs under needs to have access to the Nominatim database. You can find the user like [this](https://serverfault.com/questions/125865/finding-out-what-user-apache-is-running-as), for default Ubuntu operating system for example it's `www-data`.