It would be nice to have an option to present the current status/priority/client/assigned on top of the ticket presentation.

The attached patch creates such an option. Please consider applying it.

To apply this patch cleanly you should first apply the patch available at #975954: Create an option to allow default client for new tickets to be empty.

CommentFileSizeAuthor
#8 support-create_option_to_present_current_status_on_top_2.patch4.67 KBWergles
#4 support-create_option_to_present_current_status_on_top_2.patch4.7 KBrsevero
#2 support-create_option_to_present_current_status_on_top_3.patch4.7 KBrsevero
#1 support-create_option_to_present_current_status_on_top_2.patch4.7 KBrsevero
support-create_option_to_present_current_status_on_top.patch5.27 KBrsevero

Comments

rsevero’s picture

rsevero commented
StatusFileSize
new support-create_option_to_present_current_status_on_top_2.patch4.7 KB

Please disregard the previous patch as it has unnecessary SQL queries.

Here is the improved version of it.

rsevero’s picture

rsevero commented
StatusFileSize
new support-create_option_to_present_current_status_on_top_3.patch4.7 KB

I hope this is the final version...

rsevero’s picture

rsevero commented
StatusFileSize
new support-create_option_to_present_current_status_on_top_2.patch4.7 KB

Maybe working in a Sunday afternoon isn't such a good idea. Your start to get confused.

The right patch is number two. I'm re-attaching it here to try to lessen the confusion I'm making.

Patch number 3 has a parameter passed as reference without real need.

jeremy’s picture

jeremy commented
Status: Needs review » Needs work

Why wouldn't we leave this up to the theme? If this can't be overridden in the theme, then that in itself is a bug.

rsevero’s picture

rsevero commented
Status: Needs work » Needs review

I believe this same feature can be implemented in the theme, as almost anything can ;)

The point is that I'm not sure there would be a clean way to access _support_state, _support_priorities etc methods inside the theme.

Besides that this feature could be easily used by many more users if incorporated in the module's code.

Thanks for the great work you are doing on this module.

rsevero’s picture

rsevero commented
Version: 6.x-1.x-dev » 6.x-1.4

Please let me know if there is interest on this patch.

Wergles’s picture

Wergles commented
Version: 6.x-1.4 » 6.x-1.8
Assigned: Unassigned » Wergles
StatusFileSize
new support-create_option_to_present_current_status_on_top_2.patch4.67 KB

I've regenerated the patch against 6.x-1.8 so it gets applied with no fuzz and no offset.

Cheers.

purencool’s picture

purencool commented
Issue summary: View changes
Status: Needs review » Closed (outdated)