crawler
crawler copied to clipboard
fix "crawler:buildQueue fails with Error at offset 0 of 92 bytes"
Fix #1087
First, thanks a lot for providing the PR. To be honest. I have see this behavior before and tried to change the order of the if-checks, but gave other side effects. I don't recall from the top of my head, but some unwanted effects was present. I need to dig into this a little more again to validate the PR.
From a logical perspective, your check first sounds reasonable, as the converter is only there, is old data format is still present in queue, which shouldn't happen that often, only directly after an update.
So perhaps in Idea that I should consider implement, in the dev-main branch, as v11.x only get bugs fixes. Would be to make a command that updates the format ones, and remove the converter.
What do you think @stephan-brun @hacksch ?
PR #1117 solves this problem in a non-intrusive way by only catching the E_WARNING.
@tomasnorre you can close this PR; the problem has been solved with #1117. But please tag the latest v11.x branch commit.
Resolved in #1117