Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Blank image in html_signature when saving identity changes #4833

Closed
rcubetrac opened this issue May 28, 2015 · 4 comments
Closed

Blank image in html_signature when saving identity changes #4833

rcubetrac opened this issue May 28, 2015 · 4 comments

Comments

@rcubetrac
Copy link

Reported by v tomas on 28 May 2015 07:41 UTC as Trac ticket #1490412

Hello,

Editing an identity, if I try to attach an image (I've tried jpeg or png) with html_signature enabled and save the changes then the image that I've just attached is blank. If I inspect the image element there is no "src" attribute.

Investingating a little I've found that modifying the variable "$regexp" (line 210) from the function "rcmail_attach_images" inside the file /var/www/roundcube/program/steps/settings/save_identity.inc it works fine.

old value: '/\s(poster|src)\s*=\s*[value: '/\s(poster|src)\s*=\s*[\'"](\'"]*\S+upload-display\S+file=rcmfile([0-9]+)[\s\'"]*/'

new)*\S+upload-display\S+file=rcmfile(\w+)[\s\'"]*/'

NOTE: I'm not a PHP developer. I don't know if it's the best fix.

Environment:
OS: Debian 7.8 (with updates)
php: 5.4.39
apache: 2.2.22

Keywords: html_signature
Migrated-From: http://trac.roundcube.net/ticket/1490412

@rcubetrac
Copy link
Author

Comment by @alecpl on 28 May 2015 08:24 UTC

Confirmed.

@rcubetrac
Copy link
Author

Milestone changed by @alecpl on 28 May 2015 08:24 UTC

later => 1.1.2

@rcubetrac
Copy link
Author

Comment by @alecpl on 28 May 2015 08:35 UTC

Fixed in c044d30.

@rcubetrac
Copy link
Author

Status changed by @alecpl on 28 May 2015 08:35 UTC

new => closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant