Sub-issue for meta issue #1880976: [meta] Port examples (including submodules) to D9.4+

Problem/Motivation

D8 all the things!

Proposed resolution

Start with D7 version and figure out how to port ;)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Mile23’s picture

Issue summary: View changes
Status: Active » Needs review
FileSize
19.89 KB

Big first step. I want the testbot to chew on it.

Status: Needs review » Needs work

The last submitted patch, 1: 2102675_1.patch, failed testing.

Mile23’s picture

Status: Needs work » Needs review

1: 2102675_1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1: 2102675_1.patch, failed testing.

Mile23’s picture

Status: Needs work » Needs review

1: 2102675_1.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1: 2102675_1.patch, failed testing.

Mile23’s picture

This should pass tests. Documentation still needs to be updated, mostly removing the reliance on the tutorial.

Mile23’s picture

Status: Needs work » Needs review
Mile23’s picture

Coming together... Needs some re-write of documentation and an answer to the question of whether to re-write the tutorial it's based upon.

Mile23’s picture

FileSize
9.89 KB

Minor naming tweaks, code sniffer... Let's see how it goes.

Status: Needs review » Needs work

The last submitted patch, 10: 2102675_10.patch, failed testing.

Mile23’s picture

Status: Needs work » Needs review
FileSize
19.7 KB

Diff against wrong branch... Previous patch is actually an interdiff for this one. :-)

Status: Needs review » Needs work

The last submitted patch, 12: 2102675_12.patch, failed testing.

Mile23’s picture

Status: Needs work » Needs review
FileSize
19.86 KB

Moved simpletest_example_empty_mysql_date() to a proper class of its own so autoloading can occur.

Also coding standards.

Mile23’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

tyler36’s picture

This should be reopened.

Thanks to the "PHPUnit inititive" SimpleTest is no longer included in Drupal 9.
The testing submodule, therefore is no longer valid.

Mile23’s picture

@tyler36 Good point, simpletest is dead, long may it stay so.

Rather than re-opening this issue, perhaps file a new one to remove the simpletest example.