I've created this issue to be able to patch a D8 release of JSONlog into the 8.x branch of this module.
| Comment | File | Size | Author |
|---|
I've created this issue to be able to patch a D8 release of JSONlog into the 8.x branch of this module.
| Comment | File | Size | Author |
|---|
Comments
Comment #2
lorenzs commentedPatch attached for an initial dev release of JSONlog 8.x - to be pushed into 8.x branch.
This dev version does not include the jsonlog_test_filing() option via drush anymore. However you can do a simplified test via the admin screen.
The D8 version has been tested on our development server (PHP 7).
Comment #3
lorenzs commentedComment #4
lorenzs commentedComment #5
jacobfriis commentedComment #9
jacobfriis commented@lorenzs Truly great job you've done here :-)
However, there are several things to be done prior to a release. This is what I stumbled upon - there will turn up more on the way ;-)
Todos - major
jsonlog_default_site_id()should fetch the name of the default database via D8 means (not via D7'ish$GLOBALS['databases'], which btw is void in the context).Todos - minor
ConfigFormBase).module_load_includehas been replaced by something else ...loadInclude().Comment #10
lorenzs commentedHi Jacob;
Todos - major
Todos - minor
See patch attached with above changes.
Comment #11
lorenzs commentedTypo in services.yml - updated patch to #11
Comment #12
jacobfriis commentedComment #13
jacobfriis commentedOops. No wonder it didn't log at my end.
Forgot about severity threshold (default: warning) - I was attempting to log debug messages.
And (more interesting) wasn't aware that the equivalent of D7
watchdog(...)D8
\Drupal::logger($type)->log(...)only logs to a single channel - Log API:
dblog was enabled too, at my end.
Comment #15
jacobfriis commented@lorenzs Great job.
Todos - major (revisited)
Todos - minor (revisited)
Added CHANGELOG.txt
Great as easily accessible tracking. And useful when creating release(note)s.
Comment #16
lorenzs commentedComment #17
lorenzs commentedBeta release for D8 is now available for testing.
https://www.drupal.org/project/jsonlog/releases/8.x-1.0-beta1
Comment #19
lorenzs commented8.x-1.0 released.