Review Request 129992: Fix save image/sound files to relative path in kvtml2writer

Xin Huang xinhuang.abc at gmail.com
Thu Mar 23 23:34:36 UTC 2017


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129992/
-----------------------------------------------------------

(Updated March 23, 2017, 5:34 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Edu.


Changes
-------

Submitted with commit c70c1433528bc75e934960dd114dd8beae626656 by Albert Astals Cid on behalf of Xin Huang to branch Applications/17.04.


Repository: libkeduvocdocument


Description
-------

```
bool KEduVocKvtml2Writer::writeTranslation( QDomElement &translationElement, KEduVocTranslation* translation )
{
    // ...
    // sound
    if ( !translation->soundUrl().isEmpty() ) {
        QString urlString;
        qDebug() << "soundUrl" << translation->soundUrl();
        if ( KIO::upUrl(m_doc->url().adjusted(QUrl::RemoveFilename)).isParentOf( translation->soundUrl()) ) {
            // try to save as relative url
            urlString = m_doc->url().toString(QUrl::RemoveFilename) + '/' + translation->soundUrl().toString();     // <------------ HERE
        } else {
            urlString =  translation->soundUrl().url();
        }
        qDebug() << "urlString" << urlString;
        translationElement.appendChild( newTextElement( KVTML_SOUND, urlString ) );
    }
```

Notice the line marked: The soundUrl of a translation parley passed in is "file:///path/to/kvtml/kvtmlfilename_files/File:En-us-Word.ogg", and the `m_doc->url()` is "file:///path/to/kvtml/kvtmlfilename.kvtml". By the old logic saving as relative url, the result will be "file:///path/to/kvtml/file:///path/to/kvtml/kvtmlfilename_files/Files:En-us-word.ogg".


Diffs
-----

  keduvocdocument/keduvockvtml2writer.cpp ff9c292 

Diff: https://git.reviewboard.kde.org/r/129992/diff/


Testing
-------


Thanks,

Xin Huang

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20170323/8160cbf2/attachment.html>


More information about the kde-edu mailing list