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.
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff-2889352-2-4.txt | 759 bytes | naveenvalecha |
#4 | 2889352-4.patch | 2.25 KB | naveenvalecha |
Comments
Comment #2
naveenvalechaHere we go
//Naveen
Comment #3
jibranWhy do we need this?
Comment #4
naveenvalecha@jibran,
we don't need this :) Removed this. Thanks for your review.
//Naveen
Comment #5
jibranThanks!
Comment #6
dawehnerMaybe a child class want's to defer the importing to do some questions, but yeah I guess in this case we don't need it.
Comment #7
Gábor HojtsyHm, we did not do this before, why do we need it now?
Comment #8
naveenvalechaTo fix the warning b/c ViewTestBase::setup requires the argument $import_test_views
Back to N/R
//Naveen
Comment #9
LendudeThe child class setUp() needs to match the parent, so the change is correct.
Comment #12
Gábor HojtsyMakes sense. Thanks for setting me straight :) Committed :)
Comment #13
Gábor Hojtsy