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 254923 - The image in a contact could be scaled to a sane size
The image in a contact could be scaled to a sane size
Status: VERIFIED FIXED
Product: evolution
Classification: Applications
Component: Contacts
2.2.x (obsolete)
Other All
: Normal trivial
: ---
Assigned To: Srinivasa Ragavan
Evolution QA team
Depends on:
Blocks: 310071
 
 
Reported: 2004-02-27 08:20 UTC by Mariano Suárez-Alvarez
Modified: 2013-09-10 14:00 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
resize patch (3.52 KB, patch)
2005-07-26 04:12 UTC, Srinivasa Ragavan
none Details | Review
proposed change (1.42 KB, patch)
2005-08-02 12:51 UTC, Frank Arnold
none Details | Review

Description Mariano Suárez-Alvarez 2004-02-27 08:20:05 UTC
Setting the image in a contact from a uncropped picture, which is what
you'd get from a camera, for example, ends up with a huge pic in the
preview pane for that contact. Maybe the image could be scaled into a sane
size?
The definition of sane in this context is, of course, left as an exercice
for the reader.
Comment 1 Hans Petter Jansson 2004-05-20 01:16:22 UTC
Seems to scale fine in current build... Can you try it with a more
recent Evolution and let me know how goes?
Comment 2 André Klapper 2005-06-04 09:18:51 UTC
it does NOT scale in 2.2.2, note that this is about the contact *preview pane*.

retargetting.
Comment 3 Srinivasa Ragavan 2005-07-26 04:12:57 UTC
Created attachment 49757 [details] [review]
resize patch

It prompts for resize and stores the reqd size
Comment 4 Srinivasa Ragavan 2005-07-28 11:39:39 UTC
Commited to cvs
Comment 5 Frank Arnold 2005-08-02 12:50:35 UTC
As pointed out by Åsmund Skjæveland on gnome-i18n list:

The added error message needs some work from i18n side. At first it should be
marked for translation, and there should be no trailing space.
Comment 6 Frank Arnold 2005-08-02 12:51:54 UTC
Created attachment 50131 [details] [review]
proposed change
Comment 7 Srinivasa Ragavan 2005-08-03 05:06:35 UTC
Looks fine. Can you commit it asap?
Comment 8 Frank Arnold 2005-08-03 06:16:07 UTC
Committed.
Comment 9 André Klapper 2005-08-20 23:54:57 UTC
having a contact with a picture set, i wanted to verify that bug 254923 has been
fixed by srini.
choosing the contact from the list and having the preview pane open, i already
saw that the photo wasn't displayed and only the normal "red cross on a paper"
icon that gnome normally shows if a picture has been removed or isn't found. i
don't know which photo i chose; perhaps i've really moved it to another
directory or anything else. so i wanted to choose a new one. but after opening
the contact editor and clicking on the picture icon (it was displayed as if i
would have never ever set a photo for that contact, evolution freezes.
this is evolution-2.3.7.0.200508170145-0.snap.novell.11.1 on suse9.3.
if i try to set a picture on a contact i hadn't set a picture before to, the
same happens (same stacktrace too). :-(

andre@embrace:~> gdb evolution
GNU gdb 6.3
Copyright 2004 Free Software Foundation, Inc.
GDB is free software, covered by the GNU General Public License, and you are
welcome to change it and/or distribute copies of it under certain conditions.
Type "show copying" to see the conditions.
There is absolutely no warranty for GDB.  Type "show warranty" for details.
This GDB was configured as "i586-suse-linux"...Using host libthread_db library
"/lib/tls/libthread_db.so.1".

(gdb) run
Starting program: /opt/gnome/bin/evolution

[Thread debugging using libthread_db enabled]
[New Thread 1097789056 (LWP 10827)]
[New Thread 1105783728 (LWP 10840)]
adding hook target 'source'
[New Thread 1113504688 (LWP 10842)]
[New Thread 1115605936 (LWP 10843)]

(evolution:10827): Gdk-CRITICAL **: gdk_gc_set_foreground: assertion `GDK_IS_GC
(gc)' failed
[New Thread 1117707184 (LWP 10846)]
[Thread 1117707184 (LWP 10846) exited]

(evolution:10827): libebook-CRITICAL **: e_contact_get_const: assertion
`field_id >= 1 && field_id <= E_CONTACT_LAST_SIMPLE_STRING' failed
(evolution:10827): libebook-CRITICAL **: e_contact_get_const: assertion
`field_id >= 1 && field_id <= E_CONTACT_LAST_SIMPLE_STRING' failed
(evolution:10827): libebook-CRITICAL **: e_contact_get_const: assertion
`field_id >= 1 && field_id <= E_CONTACT_LAST_SIMPLE_STRING' failednew dimensions
= (48,48)
initial setting of an image.  no scaling
scale = 1

Program received signal SIG33, Real-time event 33.
[Switching to Thread 1115605936 (LWP 10843)]
0xffffe410 in ?? ()
(gdb) thread apply all bt

Thread 2 (Thread 1105783728 (LWP 10840))

  • #0 ??
  • #1 ??
  • #2 ??
  • #3 ??
  • #4 pthread_cond_timedwait
    from /lib/tls/libpthread.so.0
  • #5 _wapi_handle_wait_signal_handle
    from /usr/lib/libmono.so.0
  • #6 WaitForSingleObjectEx
    from /usr/lib/libmono.so.0
  • #7 ves_icall_System_GC_WaitForPendingFinalizers
    from /usr/lib/libmono.so.0
  • #8 ves_icall_System_Threading_Thread_Thread_internal
    from /usr/lib/libmono.so.0
  • #9 _wapi_timed_thread_exit
    from /usr/lib/libmono.so.0
  • #10 GC_start_routine
    from /usr/lib/libmono.so.0
  • #11 start_thread
    from /lib/tls/libpthread.so.0
  • #12 clone
    from /lib/tls/libc.so.6

Thread 1 (Thread 1097789056 (LWP 10827))

  • #0 ??
  • #1 ??
  • #2 ??
  • #3 ??
  • #4 __nptl_setxid
    from /lib/tls/libpthread.so.0
  • #5 seteuid
    from /lib/tls/libc.so.6
  • #6 _gnome_vfs_get_daemon_force_probe_callback
    from /opt/gnome/lib/libgnomevfs-2.so.0
  • #7 gnome_vfs_method_get
    from /opt/gnome/lib/libgnomevfs-2.so.0
  • #8 gnome_vfs_uri_new_private
    from /opt/gnome/lib/libgnomevfs-2.so.0
  • #9 gnome_vfs_url_show
    from /opt/gnome/lib/libgnomevfs-2.so.0
  • #10 gnome_vfs_make_uri_canonical
    from /opt/gnome/lib/libgnomevfs-2.so.0
  • #11 fs_module_init
    from /opt/gnome/lib/gtk-2.0/2.4.0/filesystems/libgnome-vfs.so
  • #12 fs_module_init
    from /opt/gnome/lib/gtk-2.0/2.4.0/filesystems/libgnome-vfs.so
  • #13 gtk_file_system_filename_to_path
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #14 fs_module_create
    from /opt/gnome/lib/gtk-2.0/2.4.0/filesystems/libgnome-vfs.so
  • #15 g_type_create_instance
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #16 g_object_class_override_property
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #17 g_object_newv
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #18 g_object_new_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #19 g_object_new
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #20 gtk_file_system_gnome_vfs_new
    from /opt/gnome/lib/gtk-2.0/2.4.0/filesystems/libgnome-vfs.so
  • #21 fs_module_create
    from /opt/gnome/lib/gtk-2.0/2.4.0/filesystems/libgnome-vfs.so
  • #22 gtk_file_selection_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #23 gtk_file_selection_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #24 gtk_file_chooser_dialog_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #25 g_object_freeze_notify
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #26 g_object_class_override_property
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #27 gtk_file_chooser_dialog_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #28 g_object_newv
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #29 g_object_new_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #30 g_object_new
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #31 gtk_file_chooser_dialog_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #32 gtk_file_chooser_widget_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #33 g_object_newv
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #34 g_object_new_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #35 g_object_new
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #36 gtk_file_chooser_dialog_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #37 g_object_newv
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #38 g_object_new_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #39 g_object_new
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #40 gtk_file_chooser_dialog_get_type
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #41 gtk_file_chooser_dialog_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #42 image_clicked
    at e-contact-editor.c line 2655
  • #43 g_cclosure_marshal_VOID__VOID
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #44 g_closure_invoke
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #45 g_signal_chain_from_overridden
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #46 g_signal_emit_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #47 g_signal_emit
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #48 gtk_button_clicked
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #49 gtk_button_new
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #50 g_cclosure_marshal_VOID__VOID
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #51 g_closure_ref
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #52 g_closure_invoke
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #53 g_signal_chain_from_overridden
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #54 g_signal_emit_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #55 g_signal_emit
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #56 gtk_button_released
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #57 gtk_button_released
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #58 gtk_marshal_BOOLEAN__VOID
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #59 g_closure_ref
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #60 g_closure_invoke
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #61 g_signal_chain_from_overridden
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #62 g_signal_emit_valist
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #63 g_signal_emit
    from /opt/gnome/lib/libgobject-2.0.so.0
  • #64 gtk_widget_get_default_style
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #65 gtk_propagate_event
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #66 gtk_main_do_event
    from /opt/gnome/lib/libgtk-x11-2.0.so.0
  • #67 gdk_add_client_message_filter
    from /opt/gnome/lib/libgdk-x11-2.0.so.0
  • #68 g_main_context_dispatch
    from /opt/gnome/lib/libglib-2.0.so.0
  • #69 g_main_context_acquire
    from /opt/gnome/lib/libglib-2.0.so.0
  • #70 g_main_loop_run
    from /opt/gnome/lib/libglib-2.0.so.0
  • #71 bonobo_main
    from /opt/gnome/lib/libbonobo-2.so.0
  • #72 main
    at main.c line 602
  • #0 ??
The program is running.  Exit anyway? (y or n) y
Comment 10 Srinivasa Ragavan 2005-08-22 03:50:58 UTC
Oh. This looks like the same gnome-vfs hang. Andre, just try opening any
fileselector in evo this hang will happen. R-Click on attachments also hang. If
so its some another bug. Can you please check it ?
Comment 11 André Klapper 2005-08-22 11:33:33 UTC
srini: ah, good to know. yeah, it's the same gnome-vfs hang. is it filed
anywhere so i can mark bugs as duplicates (e.g. my bug 314096)?

closing this again, going to verify this after gnome-vfs hang has been fixed.
Comment 12 André Klapper 2005-08-27 13:07:21 UTC
running evolution-2.3.8.0.200508260453-0.snap.novell.11.1, i get the question
pop up "The image you have selsected is large. Do you want to resize and store
it?". if i choose "no", the contact editor window becomes read-only and
everything is greyed out. i *can* close it, but it takes a few seconds...
if i choose yes, it becomes a small one (48x48px or whatever the shell told me),
but when i saw the question for the first time, i asked myself "will my original
picture be made smaller or just a copy of it stored in the addressbook
database?" perhaps there should be a hint "The original image will not be
changed/altered." or something like that.
but perhaps i'm a user who just tries too hard to be a stupid user sometimes... :-)


anyway, verifying this is fixed.