After deleting the last item of the queue I get two errors:

Notice: Undefined variable: result in-QueueUISystemQueue->inspect() (from line 52 -/site-name/sites/all/modules/queue_ui/lib/QueueUISystemQueue.php).

and:

Warning: Invalid argument supplied for foreach() in-QueueUISystemQueue->inspect() (line from -site-name/sites/all/modules/queue_ui/lib/QueueUISystemQueue.php).

CommentFileSizeAuthor
#3 queue_ui-redirect_empty_queue-2227897-3.patch477 bytesjustinstandring
#2 undefined_array-2227897-2.patch370 bytessoul88

Issue fork queue_ui-2227897

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

taitai’s picture

taitai commented
Title: Deleting last item not processed property » Deleting last item not processed properly
soul88’s picture

soul88
Primary language Ukrainian
Location Ukraine
commented
StatusFileSize
new undefined_array-2227897-2.patch370 bytes

Please check if this helps

justinstandring’s picture

justinstandring commented
Status: Active » Needs review
StatusFileSize
new queue_ui-redirect_empty_queue-2227897-3.patch477 bytes

#2 patch works, but leaves the user at the inspect page with an empty table. This patch (#3) redirects them back to the list of queues.

mouhammed’s picture

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

The patch #3 works.

belaustegui’s picture

belaustegui commented

I can confirm that the patch #3 works perfectly. Will it be merged into the stable version?
Thank you!

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
commented

Bump.

jimmyko’s picture

jimmyko commented

+1 for the patch.

ronaldmulero’s picture

ronaldmulero commented

#3 works for me
• queue_ui 7.x-2.x

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
commented

Cool. We've got more validated reports patch #3 is good. Let's move this in. :)

voleger made their first commit to this issue’s fork.

voleger opened merge request !5

  • voleger committed 7a08bf5 on 7.x-2.x 
    Issue #2227897 by justinstandring, Soul88, taitai, Chris Charlton,...
voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented

Pushed into 7.x-2.x
Thanks

voleger’s picture

voleger
he/his
Primary language Ukrainian
Location Ukraine, Rivne
commented
Status: Reviewed & tested by the community » Fixed

Set proper status.

Status: Fixed » Closed (fixed)

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