Hi,
I've come across what appears to be a bug with the module that renders it unusable on my site.
When I install the module in the UI, I get a white screen with "The website encountered an unexpected error. Please try again later."
I can press the 'Back' button and get back to the UI. However, the menu option as described in the module description is not there, and the logs do not report that the module was installed, although I am able to uninstall it. Instead, the following error appears in the logs:
Symfony\Component\DependencyInjection\Exception\LogicException: Service 'logger.cslog' for consumer 'logger.factory' does not implement Psr\Log\LoggerInterface. in Drupal\Core\DependencyInjection\Compiler\TaggedHandlersPass->process() (line 126 of /Users/jess/Sites/devdesktop/message-lab/core/lib/Drupal/Core/DependencyInjection/Compiler/TaggedHandlersPass.php).
The same error appears if I try to run drush cr
at the top of a stack trace. If I 'uninstall' the module, I can run drush cr
without issue.
Flushing the caches from within the UI makes no difference.
I'd really like to use this module so if anyone has any suggestions please let me know.
Thanks,
Jesse.
Comment | File | Size | Author |
---|---|---|---|
#23 | __content syn2.png | 44.51 KB | Londova |
#23 | __content syn1.png | 16.84 KB | Londova |
#20 | __error.png | 20.71 KB | Londova |
#14 | content_sync_service_cslog_debug.patch | 388 bytes | JonasSavs |
Comments
Comment #2
imperator_99 CreditAttribution: imperator_99 commentedPlease not that this error also occurs with the dev version.
EDIT: It also appears that only removing it entirely from the file system stops the error from reappearing.
Comment #3
Blanca.Esqueda CreditAttribution: Blanca.Esqueda as a volunteer and at Portage CyberTech commentedHi Jesse,
I wasn't able to replicate it. I even used https://simplytest.me with content_sync and didn't have that issue either.
If you can let me know a bit more info, maybe some other modules installed that use the logs.. I could do more troubleshooting and resolve any conflict.
FYI a new version is being developed right know, and in fact one of the improvements is to use a custom log. It should be available soon.
Regards,
Blanca
Comment #4
Blanca.Esqueda CreditAttribution: Blanca.Esqueda as a volunteer and at Portage CyberTech commentedHi Jesse,
Again me, even when I've not been able to replicate the error I can see that it is related to the logs.
Please, try commenting out (add # at the beginning of each line) or deleting the following lines:
from this file:
content_sync.services.yml
Note: the above would remove the log functionality (summary of import, export,warnings, errors) of the module, but you should be able to use the rest of the functionality. Export your site content first, before any import so even if you don't have logs at least you would have your original content for reference.
Please let me know if this works for you,
Blanca
Comment #5
imperator_99 CreditAttribution: imperator_99 commentedThanks Blanca, that's done the trick. I was able to install the module and it all appears to be working fine. The only issue I had further was an error about a missing theme, which I presume is related to commenting out the code as you suggested, but it doesn't seem to have had any effect beyond the error message.
Cheers,
Jesse.
Comment #6
bojanz CreditAttribution: bojanz at Centarro commentedSame error just crashed the fresh Drupal 8.3 install I wanted to use to test this module.
Removed the service to continue testing.
Comment #7
lamp5The same error on Drupal 8.3.
Comment #8
Blanca.Esqueda CreditAttribution: Blanca.Esqueda as a volunteer and at Portage CyberTech commentedFYI, we were able to replicate the issue.
Next release is planned for Aug 4th, it would replace the service and include other features as Drush commands and support for paragraphs, etc..
Would update this issue as soon as the next release is done!
Comment #9
lalop CreditAttribution: lalop commentedAny news of the new release ?
Comment #10
Blanca.Esqueda CreditAttribution: Blanca.Esqueda commentedHi lalop,
More contributors are joining the content_sync team, we are merging work it is why the next release it is taking us a bit longer.
On the bright side more people involved means more features being added to the release, plus more help to resolve future issues faster.
I'm afraid to say a date, because we are delayed already of our expected release date, but we hope that the new release would be before the end of August.
Thank you for your patience.
Comment #11
imperator_99 CreditAttribution: imperator_99 commentedClosing this issue as the maintainer looks to be fixing it in the next release.
Comment #12
websiteworkspace CreditAttribution: websiteworkspace commented@imperator_99
The proposed release did not occur in August 2017.
The problem described here with this module still exists.
Comment #13
websiteworkspace CreditAttribution: websiteworkspace commentedComment #14
JonasSavs CreditAttribution: JonasSavs as a volunteer and commentedHi all,
here a patch to fix it.
Looks like there is a bad encoded character. The g to be exact. Just erase the ContentSyncLo(g) and re-add it or apply this patch.
Regards
Comment #15
c-c-m CreditAttribution: c-c-m commentedI can confirm the same problem with a fresh install (drupal 8.4.1) and Content_sync 8.x-1.2.
Applied patch at #14 worked fine and allowed me to install the module.
Comment #17
Blanca.Esqueda CreditAttribution: Blanca.Esqueda commentedThank you @JonasSavs,
I completely missed that bad encoded char.
I already added the fix to the the new version 8.x-2.x and credited you:
https://www.drupal.org/node/2865869/committers
Please feel free to pull version 8.x-2.x and do some testing, but be aware that the yaml files are different from one version to the other as some changes were needed to implement new functionality. (that means the same version has to run on the source and target sites).
Comment #18
Londova CreditAttribution: Londova commentedThe following error blocks the module installation on existing Drupal Commerce site:
"Drupal\Core\Entity\Exception\UndefinedLinkTemplateException: No link template 'canonical' found for the 'commerce_log' entity type in Drupal\Core\Entity\Entity->toUrl() (line 215 of /.../core/lib/Drupal/Core/Entity/Entity.php)."
I uninstalled the "commerce_log" module and repeat the procedure. A similar error occurred:
"Drupal\Core\Entity\Exception\UndefinedLinkTemplateException: No link template 'canonical' found for the 'commerce_order_item' entity type in Drupal\Core\Entity\Entity->toUrl() (line 215 of /.../core/lib/Drupal/Core/Entity/Entity.php)."
Please, help.
Comment #19
websiteworkspace CreditAttribution: websiteworkspace commented@londova
You need to make the following change to the module's code until a new release is available.
Comment #20
Londova CreditAttribution: Londova commented@websiteworkspace
I made this change before.
The actual error appear when enabling the module.
See the attached image for error message.
Comment #21
Blanca.Esqueda CreditAttribution: Blanca.Esqueda commentedHi Londova,
Do you have the same issue if you the version 8.x-2.x?
There is a branch that has that version.
Please let me know,
Blanca
Comment #22
Londova CreditAttribution: Londova commented@Blanca.Esqueda
Drupal 8.4.2; Content Synchronization 8.x-1.2; Drupal Commerce 8.x-2.x-dev
Comment #23
Londova CreditAttribution: Londova commentedVery strange behaviour.
Yesterday "switch ON" the module, I had the error from picture "__error.png".
Today morning I restart the computer and my screen displayed absolutely other messages - see pictures attached.
Is that as should be?
Comment #24
Londova CreditAttribution: Londova commentedI disabled the module, cleared cache and enable it again.
The error message appear again.
Comment #25
xrampage16 CreditAttribution: xrampage16 commentedJust performed install of 8.x-1.2, and received the error. I performed the same action as #19, and did an uninstall, and went back and performed an install, and it resolved the issue.
Comment #26
Blanca.Esqueda CreditAttribution: Blanca.Esqueda commentedPlease use the version 8.2:
https://www.drupal.org/project/content_sync/releases/8.x-2.x-dev
This is a dev version but after adding the option to handle images/files as well this would be the stable version and the one we would be maintaining. Be aware that the yaml files are different from one version to the other as some changes were needed to implement new functionality (that means the same version has to run on the source and target sites).
Comment #27
jasonellington CreditAttribution: jasonellington as a volunteer commentedVery helpful. thanks.
Comment #28
Blanca.Esqueda CreditAttribution: Blanca.Esqueda at Portage CyberTech commentedPatch has been included to release 8.x-2.0
Comment #29
Blanca.Esqueda CreditAttribution: Blanca.Esqueda at Portage CyberTech commented