[GEOCODER_45] : Pull request : composer, sql, openstreetmap_activation

[heitzjm]

This PR tries to bring some adjustements (at least, I hope so) :

• sql : it seems that mysql does not recognize string type, so I changed it to text
• postInstall : needed to activate OpenStreetMap during a fresh install (see #45 )
• composer.json : while trying to install the extension altogether with drupal 9 and civicrm, I had to remove the replace section, because it did not install at all the required libraries mentionned in the replace section. Moreover, I tried to switch to Drupal 10, which leads to PHP 8.1 and guzzle v7, and requiring (indirectly) guzzle v6 make installation impossible. Using php-http/discovery without allowing plugin to run seems to provide a convenient way for solving this problem, as geocoder seems only need a PSR18 http client (running the plugin changes composer.json)
• I did not add the resulting vendor directory, as it should be produced by composer : however, this might be needed if someone install the extension directly and is relying on the included vendor directory (I guess it depends on the way an extension is packaged/released - I have no experience on that point)

I didn't manage to run the already included test suite ; however, I was able to run the unit test file (UpgraderTest.php) I added by specifying the test file on phpunit command line.

I did my best for this PR, but I'm still a novice in CiviCRM and in extensions, so please tell me if something is wrong or not appropriate.

Thanks !

[mattwire]

@heitzjm Can you split this into separate PRs? For example the composer changes seem more complex and will need to be reviewed separately

[heitzjm]

@mattwire , @eileenmcnaughton I split this PR into three separate PRs : #48 , #49 , #50 .

[github-actions[bot]]

Stale pull request message