Issue Details (XML | Word | Printable)

Key: MDL-17542
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Francois Marier
Reporter: Francois Marier
Votes: 0
Watchers: 2
Operations

Add/Edit UI Mockup to this issue
If you were logged in you would be able to see more operations.
Moodle

html2text appears to be incompatible with the GPL

Created: 07/Dec/08 04:59 AM   Updated: 16/Jun/09 10:01 PM
Return to search
Component/s: Lib
Affects Version/s: 1.6.8, 1.7.6, 1.8.7, 1.9.3
Fix Version/s: 1.8.9, 1.9.5

File Attachments: 1. File html2text.new.php (20 kB)
2. File html2text.php (15 kB)
3. Text File html2text_replacement18.patch (28 kB)
4. Text File html2text_replacement19.patch (28 kB)
5. Text File html2text_replacement20.patch (28 kB)

Issue Links:
Dependency
 
Relates
 

URL: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=507947
Participants: Anthony Borrow, Dongsheng Cai, Eloy Lafuente (stronk7), Francois Marier, Helen Foster, Martin Dougiamas and Petr Skoda
Security Level: None
QA Assignee: Helen Foster
Difficulty: Moderate
Resolved date: 22/May/09
Affected Branches: MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
Fixed Branches: MOODLE_18_STABLE, MOODLE_19_STABLE


 Description  « Hide
This library was reported in http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=507947

The license for html2text is not included in the code, but rather on the website of the author:

  http://www.howtocreate.co.uk/jslibs/termsOfUse.html

The most problematic bit of html2text's license is the requirement to get written permission to distribute:

"However, if you want to include the scripts [...] in bundled or distributed scripts/software (such as Opera or Dashboard widgets, or the Formativ Academic Timetable), you should contact me first and obtain my express written permission."

Then, there is a bit of discrimination for commercial sites through the addition of an "advertising clause":

"Commercial/profit-making public websites:

You should put a note saying that the script was provided for free by http://www.howtocreate.co.uk.

You put a plain text (ie. not script) link to my site on your policy page or other appropriate page, to help me with my search engine rankings."

The Debian developer who filed the bug suggested that we take a look at replacing this library with the one I have attached here. It's got the same name but is under the GPLv2+.

Based on a quick 1.9 grep, that file is only used in lib/weblib.php:html_to_text(), which in turn is only used in format_text_email() of the same library file.

 All   Comments   Change History   Version Control      Sort Order: Ascending order - Click to sort in descending order
Petr Skoda made changes - 07/Dec/08 05:47 AM
Field Original Value New Value
Fix Version/s 1.9.4 [ 10300 ]
Fix Version/s 2.0 [ 10122 ]
Eloy Lafuente (stronk7) made changes - 10/Dec/08 03:48 AM
Assignee moodle.com [ moodle.com ] Petr ?koda [ skodak ]
Francois Marier made changes - 16/Dec/08 02:05 PM
Attachment html2text_replacement.patch [ 15888 ]
Martin Dougiamas made changes - 16/Dec/08 02:39 PM
Assignee Petr ?koda [ skodak ] Dongsheng Cai [ dongsheng ]
Dongsheng Cai made changes - 16/Dec/08 05:20 PM
Attachment html2text.php [ 15889 ]
Francois Marier made changes - 17/Dec/08 10:06 AM
Attachment html2text.new.php [ 15897 ]
Francois Marier made changes - 17/Dec/08 11:34 AM
Attachment html2text.inc [ 15825 ]
Francois Marier made changes - 17/Dec/08 11:34 AM
Attachment html2text_replacement.patch [ 15888 ]
Francois Marier made changes - 17/Dec/08 11:35 AM
Attachment html2text_replacement19.patch [ 15898 ]
Francois Marier made changes - 17/Dec/08 11:36 AM
Attachment html2text_replacement18.patch [ 15899 ]
Francois Marier made changes - 17/Dec/08 11:38 AM
Attachment html2text_replacement20.patch [ 15900 ]
Francois Marier made changes - 17/Dec/08 11:56 AM
Fix Version/s 1.9.4 [ 10300 ]
Assignee Dongsheng Cai [ dongsheng ] Francois Marier [ francois ]
Status Open [ 1 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Anthony Borrow made changes - 25/Feb/09 03:12 PM
Link This issue will help resolve MDL-18368 [ MDL-18368 ]
Anthony Borrow made changes - 25/Feb/09 03:15 PM
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Francois Marier made changes - 03/Apr/09 07:17 AM
Fix Version/s 1.8.9 [ 10322 ]
Resolution Fixed [ 1 ]
Fix Version/s 1.9.5 [ 10320 ]
Status Reopened [ 4 ] Resolved [ 5 ]
Fix Version/s 2.0 [ 10122 ]
Petr Skoda made changes - 21/May/09 06:43 PM
Link This issue has been marked as being related by MDL-19266 [ MDL-19266 ]
Petr Skoda made changes - 21/May/09 06:44 PM
Status Resolved [ 5 ] Reopened [ 4 ]
Resolution Fixed [ 1 ]
Francois Marier made changes - 22/May/09 10:11 AM
Resolution Fixed [ 1 ]
Status Reopened [ 4 ] Resolved [ 5 ]
Petr Skoda made changes - 10/Jun/09 03:23 AM
Link This issue has been marked as being related by MDL-19447 [ MDL-19447 ]
Helen Foster made changes - 16/Jun/09 10:01 PM
Status Resolved [ 5 ] Closed [ 6 ]
QA Assignee tsala