Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Support of Beans
see #1390872 for more information
At first need implement patch integration-independent-modules-1390872-15.patch
Comment | File | Size | Author |
---|---|---|---|
#1 | bean-integration-2127731-0.patch | 17.03 KB | Samvel |
Comments
Comment #1
Samvel CreditAttribution: Samvel commentedComment #2
Samvel CreditAttribution: Samvel commentedComment #3
DJMyles CreditAttribution: DJMyles as a volunteer commentedIs there an update as to whether this patch will be incorporated into linkchecker? We have tested this locally and it has passed our internal integration and UAT testing. We have patched our production branch with it as we use URL rich beans extensively throughout our site.
Comment #4
Delphine Lepers CreditAttribution: Delphine Lepers at Trasys for European Commission and European Union Institutions, Agencies and Bodies commentedI have the same remarks and the same question ! :)
Comment #5
Samvel CreditAttribution: Samvel at DrupalJedi commentedUp!
Comment #6
afinnarn CreditAttribution: afinnarn at University of Colorado Boulder commentedI've used this patch for awhile but found a potential bug today. On line 191 in "_linkchecker_bean_add_bean_links()", the "db_insert()" caused a PDO exception since there was a duplicate bid/lid combo already in there.
Before debugging too much, I thought, "why not change it to db_merge()?" which performs either an insert or update query. I've uploaded my lazy one line change...not doing an interdiff for this. Ought to probably add exception handling, but with db_merge(), I wouldn't be too concerned.
Comment #7
afinnarn CreditAttribution: afinnarn at University of Colorado Boulder commentedI'm marking this back to needs work since I had a fatal error with it, but feel free to refute me and say it's only me. I'd like this patch to go in since I've only had this issue once.
Comment #8
afinnarn CreditAttribution: afinnarn at University of Colorado Boulder commentedOkay, here is my non-lazy patch that still works for me. The issue I had is very ephemeral and hard to catch, but adding the merge query and wrapping in a try/catch block seemed appropriate.
Comment #9
afinnarn CreditAttribution: afinnarn at University of Colorado Boulder commentedComment #10
afinnarn CreditAttribution: afinnarn at University of Colorado Boulder commentedscrew it, just use the first patch. My issue is with a slave database and using
db_ignore_slave()
but I doubt you'd want to guard against a slow slave db since most sites don't setup a master/slave configuration.Comment #11
kala4ekJust to not confuse anyone - Need to use the patch from #1.
Other (wrong) patches were hidden.
Comment #13
VladimirAusThank you for contributions. Committed! 🥂