Os X Yosemite Qt drag and drop file name bug

Question

I was trying to use a simple drag and drop feature in my Qt app. Here is my code:

MyWindow::MyWindow(QWidget *parent)
{
    ..........
    setAcceptDrops(tru         


        

Answer 1

To apply the patch, you would need to download the Qt Sources (ex. use git) and then add the lines needed to retrieve the filepath instead of the fileId using the:

QUrl::fromNSURL([url.toNSURL() filePathURL]);

method.

Here is the part where it should be added: Patch Code

When you did this, you need to build Qt and build your project with this patched version to take use of the changes you made.

Answer 2

Right, this a generic

How do I patch the Qt source in order to fix a bug or add a feature?

request. This may be useful to document in general, therefore here goes the answer.

You could grab the official release tarball and patch that without git or you can go through the repository. I would personally opt for the second since patching and cherry-picking with git is easier in my humble opinion. These are the steps that you need to take:

1. Clone the Qt 5 repository

   git clone git://gitorious.org/qt/qt5.git qt5
   

2. Go to the cloned directory

   cd qt5
   

3. Initialize the repository

   perl init-repository
   

4. Go to the qtbase directory which you need to patch

   cd qtbase
   

5. Create a gerrit account if you have none yet. This step is optional.

6. a. Fetch and cherry-pick the fix from Gerrit

    git fetch https://yourgerritusername@codereview.qt-project.org/qt/qtbase refs/changes/11/92611/4 && git cherry-pick FETCH_HEAD

b. Do not create a gerrit account

This will be feasible in this case since it is a very minor change to the source code, and all the rest is just change to the benchmarks. There are no expected updates to the craft of the change, either.

    * git apply
    * copy and paste the following snippet to the standard input

    commit 66a305f282e33b1bf12bec21f416c8ba6730cd40
    Author: Cyril Oblikov <munknex@gmail.com>
    Date:   Tue Aug 19 16:18:25 2014 +0300

        OSX: convert file reference url to path-based url

        According to Apple's spec URL can be:
        path-based URL: file://localhost/Users/steve/Documents/MyFile.txt
        or
        file reference URL: file:///.file/id=6571367.2773272/

        On OSX 10.10 file reference urls are copied to clipboard during Drag&Drop.

        This patch converts file reference url to path-based url.

        Comment on performance: converting 1000 urls on Macbook Air 2013 takes
        about 15 ms.

        Also benchmark is added.

        Change-Id: Ia42386cd90d1eb11d04ab33705f5eece6c13f370

    diff --git a/src/platformsupport/clipboard/qmacmime.mm b/src/platformsupport/clipboard/qmacmime.mm
    index 6fcd19e..2bb623f 100644
    --- a/src/platformsupport/clipboard/qmacmime.mm
    +++ b/src/platformsupport/clipboard/qmacmime.mm
    @@ -614,6 +614,8 @@ QVariant QMacPasteboardMimeFileUri::convertToMime(const QString &mime, QList<QBy
             QUrl url = QUrl::fromEncoded(data.at(i));
             if (url.host().toLower() == QLatin1String("localhost"))
             url.setHost(QString());
    +        if (url.host().isEmpty() && url.path().startsWith(QLatin1String("/.file/id=")))
    +            url = QUrl::fromNSURL([url.toNSURL() filePathURL]);
             url.setPath(url.path().normalized(QString::NormalizationForm_C));
             ret.append(url);
         }

7. Configure the project

   ./configure -developer-build -opensource -nomake examples -nomake tests
   

8. Build and install the project

   make -j4 all install
   

9. Go get some tea until it is ready