Bug 326811 – Don't click on the head track number column of a smart playlist

After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.

Bug 326811 - Don't click on the head track number column of a smart playlist


Summary:	Don't click on the head track number column of a smart playlist


Status:	RESOLVED FIXED

Product:	rhythmbox
Classification:	Other
Component:	User Interface
Version:	0.9.x
Hardware:	Other Linux

Importance:	Normal normal
Target Milestone:	---
Assigned To:	RhythmBox Maintainers
QA Contact:	RhythmBox Maintainers

URL:
Whiteboard:

Depends on:
Blocks:

Reported:	2006-01-13 10:06 UTC by Sebastien Bacher
Modified:	2006-01-17 10:27 UTC

See Also:
GNOME target:	---
GNOME version:	---

Attachments
patch (2.23 KB, patch) 2006-01-13 11:46 UTC, Jonathan Matthew	committed	Details \| Review

Description Sebastien Bacher 2006-01-13 10:06:10 UTC

This bug has been described on https://bugzilla.ubuntu.com/show_bug.cgi?id=22171

"Rhythmbox 0.9.2, dapper drake on iBook G4 1.07 GHz.

When I click on the "track" button above all numbers of each songs in a smart
playlist, it crashes my rhythmbox.

Maybe it's linked with that problem : i set my rhythmbox in order not to display
this songs number column, yet it works only with the library, not queud songs
nor a smart playlist..."


backtrace of the crash:

Backtrace was generated from '/usr/bin/rhythmbox'

Using host libthread_db library "/lib/tls/i686/cmov/libthread_db.so.1".
[Thread debugging using libthread_db enabled]
[New Thread -1227278656 (LWP 10377)]
[New Thread -1245860944 (LWP 10379)]
[New Thread -1237468240 (LWP 10378)]
0xffffe410 in __kernel_vsyscall ()

+ Trace 65175

#0 __kernel_vsyscall
#1 __waitpid_nocancel
from /lib/tls/i686/cmov/libpthread.so.0
#2 libgnomeui_segv_handle
at gnome-ui-init.c line 786
#3 <signal handler called>
#4 ??
#5 node_compare
at gsequence.c line 143
#6 g_sequence_node_insert_sorted
at gsequence.c line 786
#7 g_sequence_insert_sorted
at gsequence.c line 260
#8 rhythmdb_query_model_set_sort_order
at rhythmdb-query-model.c line 1595
#9 rb_entry_view_resort_model
at rb-entry-view.c line 2093
#10 rb_auto_playlist_source_songs_sort_order_changed_cb
at rb-auto-playlist-source.c line 455
#11 IA__g_cclosure_marshal_VOID__VOID
at gmarshal.c line 77
#12 IA__g_closure_invoke
at gclosure.c line 490
#13 signal_emit_unlocked_R
at gsignal.c line 2438
#14 IA__g_signal_emit_valist
at gsignal.c line 2197
#15 IA__g_signal_emit
at gsignal.c line 2241
#16 rb_entry_view_sync_sorting
at rb-entry-view.c line 1022
#17 rb_entry_view_column_clicked_cb
at rb-entry-view.c line 1119
#18 IA__g_cclosure_marshal_VOID__VOID
at gmarshal.c line 77
#19 IA__g_closure_invoke
at gclosure.c line 490
#20 signal_emit_unlocked_R
at gsignal.c line 2438
#21 IA__g_signal_emit_valist
at gsignal.c line 2197
#22 IA__g_signal_emit_by_name
at gsignal.c line 2265
#23 gtk_tree_view_column_button_clicked
at gtktreeviewcolumn.c line 1102
#24 IA__g_cclosure_marshal_VOID__VOID
at gmarshal.c line 77
#25 IA__g_closure_invoke
at gclosure.c line 490
#26 signal_emit_unlocked_R
at gsignal.c line 2438
#27 IA__g_signal_emit_valist
at gsignal.c line 2197
#28 IA__g_signal_emit
at gsignal.c line 2241
#29 IA__gtk_button_clicked
at gtkbutton.c line 834
#30 gtk_real_button_released
at gtkbutton.c line 1369
#31 IA__g_cclosure_marshal_VOID__VOID
at gmarshal.c line 77
#32 g_type_class_meta_marshal
at gclosure.c line 567
#33 IA__g_closure_invoke
at gclosure.c line 490
#34 signal_emit_unlocked_R
at gsignal.c line 2368
#35 IA__g_signal_emit_valist
at gsignal.c line 2197
#36 IA__g_signal_emit
at gsignal.c line 2241
#37 IA__gtk_button_released
at gtkbutton.c line 826
#38 gtk_button_button_release
at gtkbutton.c line 1262
#39 _gtk_marshal_BOOLEAN__BOXED
at gtkmarshalers.c line 83
#40 g_type_class_meta_marshal
at gclosure.c line 567
#41 IA__g_closure_invoke
at gclosure.c line 490
#42 signal_emit_unlocked_R
at gsignal.c line 2476
#43 IA__g_signal_emit_valist
at gsignal.c line 2207
#44 IA__g_signal_emit
at gsignal.c line 2241
#45 gtk_widget_event_internal
at gtkwidget.c line 3735
#46 IA__gtk_propagate_event
at gtkmain.c line 2175
#47 IA__gtk_main_do_event
at gtkmain.c line 1412
#48 gdk_event_dispatch
at gdkevents-x11.c line 2291
#49 IA__g_main_context_dispatch
at gmain.c line 1916

Comment 1 Jonathan Matthew 2006-01-13 11:46:37 UTC

Created attachment 57275 [details] [review]
patch

This just makes the track and playing icon columns unclickable.

Comment 2 James "Doc" Livingston 2006-01-17 06:44:54 UTC

Looks fine to me.

Comment 3 Jonathan Matthew 2006-01-17 10:27:28 UTC

Committed, thanks.