Currently, when a handler is missing, the user gets an error like this:

Error: handler for sparql_ep > s doesn't exist!

This is because the _broken handlers are missing from the handlers.inc implementation of hook_views_handlers(). This patch fixes that.

There was another DEBUG comment at line 212 of includes/handlers.inc. I didn't remove it because I wasn't sure whether it was really the same issue.

CommentFileSizeAuthor
#4 953718-2x.patch1.58 KBbojanz
#4 953718-3x.patch2.33 KBbojanz
missing_broken_handlers.patch2.77 KBlinclark
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Also, I found this because I was trying to #953716: Allow modules to override Broken/missing handlers.

bojanz’s picture

bojanz CreditAttribution: bojanz commented
Status: Needs review » Reviewed & tested by the community

Makes sense.
Of course, this is not needed for 7.x because we don't have hook_views_handlers() there.

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Needs work

Looks like we need a reroll. Hunk #4 fails to apply.

bojanz’s picture

bojanz CreditAttribution: bojanz commented
Status: Needs work » Reviewed & tested by the community
FileSize
953718-3x.patch2.33 KB
953718-2x.patch1.58 KB

That patch was against 3.x only (applies still, but with big offsets).

Here are two rerolls, for 2.x and 3.x.
The only difference is that 2.x has no area_broken handler (since it has no area plugins).

merlinofchaos’s picture

merlinofchaos CreditAttribution: merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Committed. THanks1

Status: Fixed » Closed (fixed)

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