Problem/Motivation

views.argument_default.schema.yml defines schema for views.argument_default.php. The corresponding plugin does not exist in core since the removal of the PHP module, so it can be removed.

The now contrib PHP module actually has a PHP argument default plugin so after fixing this is in core this issue should be moved to the PHP module queue to add the schema there.

Proposed resolution

Remove the obsolete schema.

Remaining tasks

 

Contributor tasks needed
Task Novice task? Contributor instructions Complete?
Create a patch Instructions No
Review patch to ensure that it fixes the issue, stays within scope, is properly documented, and follows coding standards Instructions No

User interface changes

-

API changes

-

Data model changes

-

Comments

tstoeckler created an issue. See original summary.

bogdan.racz’s picture

Assigned: Unassigned » bogdan.racz
bogdan.racz’s picture

StatusFileSize
new654 bytes

Added patch. Please review.

bogdan.racz’s picture

Status: Active » Needs review
bogdan.racz’s picture

Assigned: bogdan.racz » Unassigned
dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Nice! How did that ever landed given how long ago the php support was dropped.

alexpott’s picture

Category: Task » Bug report
Status: Reviewed & tested by the community » Fixed

Having schema for things that do exist is a bug.

Committed 851ae1d and pushed to 8.0.x and 8.1.x. Thanks!

  • alexpott committed 851ae1d on 8.1.x
    Issue #2623548 by rbmboogie: Remove obsolete views.argument_default.php...

  • alexpott committed 03139d9 on
    Issue #2623548 by rbmboogie: Remove obsolete views.argument_default.php...
tstoeckler’s picture

Wow, from reporting an issue to commit in under 4 hours!

Sadly d.o does not support emojis, but I am the smiley with hearts as eyes right now.

Status: Fixed » Closed (fixed)

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