I also reported this to the Roundcube Webmail developers (they are the upstream maintainer of html2text.php)...

They very quickly patched the bug this way:

479 $text = preg_replace('/&#?[a-z0-9]

;/i', '', $text);

http://trac.roundcube.net/changeset/3777

-Garret

Hi Garret
Would this also affect the display of a hyperlink in the email?

We have a forum post like this
"As of this week (Friday 30 July 2010), the Friday tutorials will be held in Computing Laboratory 6 (R G.18)."

here is the html as the words 'Computing Lab 6' and 'RG.18' are hyperlinks to maps on the website

"As of this week (Friday 30 July 2010), the Friday tutorials will be held in <strong><a href="http://www.scms.waikato.ac.nz/genquery.php?linklevel=2&linklist=SCMS_Rightbar&linkname=Computer_Laboratories&linktype=report&listby=Lab_Title&lwhere=unique_record_id=10&children=">Computing Laboratory 6</a></strong> (<a href="http://www.waikato.ac.nz/contacts/map/?term=R">R G.18</a>)."

A user is reporting that his plain text email output is:
"As of this week (Friday 30 July 2010), the Friday tutorials will be held in ()."

Hi,

Have attached three files:

Help Sessions and general admin.html
-> html we were having problems with.

Help Sessions and general admin.20100827140840.txt.withoutfix
-> using $text = preg_replace('/&[^&;]+;/i', '', $text);

Help Sessions and general admin.20100827140938.txt.withfix
-> Help Sessions and general admin.20100827140938.txt.withfix

Works for this example.

Cheers,

Troy

Script used to check: html2texttest.php

just put in admin/utils or change path to config.php

Troy

The greedy regex is still there in html2text.php in Moodle 2.0

the bug is still there in Moodle 2.0 (and latest 2.2dev as of today). A quick search shows about 10 references of the function html_to_text():

admin/cli/upgrade.php
grade/lib.php
lib/environmentlib.php
lib/weblib.php
mod/lesson/format.php
question/format.php
question/format/multianswer/format.php
question/type/questiontype.php
question/type/essay/questiontype.php

Edited to make it clear that this affects 2.x as well.

It's an easy fix.. was reported almost 2 years ago.

The RoundCubeMail developers have since updated their code again... you can see the latest here:
http://trac.roundcube.net/browser/trunk/roundcubemail/program/lib/html2text.php

Michael, this one slipped through the cracks. Probably worth adding to a sprint soon.

Bumping this issue.

Looks perfect so long as their are no entities with more than 6 characters and never will be?

Hi Raj,
i've found the relevant change that was made upstream here :

http://trac.roundcube.net/changeset?reponame=&new=4223%40trunk%2Froundcubemail%2Fprogram%2Flib%2Fhtml2text.php&old=4013%40trunk%2Froundcubemail%2Fprogram%2Flib%2Fhtml2text.php

The change was made with a couple of other changes too so this fix may be out of sync/context.

Perhaps we should consider updating the entire 'library' file here to the latest maintained copy rather than fixing the one line ?

cheers,
Aparup

Thanks Aparup,

I saw this change, but was not sure if whole file should be updated. If it's fine to do it as part of stable team then I would love to do that.

My only concern, in doing this is:

1. Is it ok to update library as part of stable work (As asked above)
2. Can this be backported to 22 and 21, I feel if not then please have this fix integrated and I will open another bug for master only.

Thanks for the report and work here guys, this has been integrated and is up for testing.

Raj: went with (2) (don't see any regressions but the changes are out of context, still - i've created MDL-31805)

Please ensure that there are no regressions, the change here does seem to fix the reported issue but the fix seems to be out of context. Note the parentheses introduced here are there because the regex was pulled from the latest version of this file. Although harmless, this simply means there are other changes to the library relevant to this latest regex.

This library is not up to date, i've created MDL-31805 to look at updating it.

Thanks Aparup

All good.
Passing!
Thanks

Well,

I wish I said it every time
you do the things you do.
You always lend a helping hand,
and I'm filled with gratitude.

You are strong and generous
for each and everyone one of us.
I am eternally grateful,
I cannot say thanks enough.

Sorry for the (un)cool bit above, lol. Closing this as fixed. Ciao