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

Evernote note stops import #253

Closed
crispinb opened this issue Nov 22, 2015 · 10 comments
Closed

Evernote note stops import #253

crispinb opened this issue Nov 22, 2015 · 10 comments

Comments

@crispinb
Copy link

The attached note (originally from the Evernote web clipper I think) will stop a Quiver import. Quiver doesn't crash, but silently stops importing with no new notebook visible in the UI. Quitting and restarting Quiver displays the new imported notebook, with all notes present up and including to this one (which is displayed sans content).

The exported note looks valid if messy in an editor, but I've only had a glance as I don't really know what to look for. (I've renamed the .enex to .txt to keep the github uploader happy)

bad_note.txt

@ylian
Copy link
Member

ylian commented Nov 22, 2015

Thanks. Yep, an exception was thrown. Should be trivial to fix.

@ylian
Copy link
Member

ylian commented Nov 22, 2015

It's due to an empty resource, no filename, no data. Might be a reminder, but not yet sure. I will fix the exception in 3.0.1.

@crispinb
Copy link
Author

It's a pretty old note (2007!). Not sure if Evernote even had reminders then. I removed it from my export file, and the same thing happened a few notes along, so maybe it was a transient issue around the time those notes were created? Anyway presumably not worth investigating far, so long as you can get Quiver to continue the import.

Having said that, I'm assuming that the errant note will be imported rather than skipped over after the fix. If not, then you probably should consider making some kind of log reporting skipped notes available to the user.

@ylian
Copy link
Member

ylian commented Nov 23, 2015

It's just en empty resource, which isn't clear to me what is for. I can just skip that resource.

@ylian
Copy link
Member

ylian commented Nov 23, 2015

Try this build: https://dl.dropboxusercontent.com/u/3445731/Quiver%203.0.1b1.zip

I added a line to skip the empty resource.

@ylian
Copy link
Member

ylian commented Nov 23, 2015

This is the empty resource:

<resource>
    <data encoding="base64"/>
    <mime>text/html</mime>
    <width>0</width>
    <height>0</height>
    <duration>0</duration>
    <resource-attributes>
        <timestamp>19700101T000000Z</timestamp>
        <attachment>true</attachment>
    </resource-attributes>
</resource>

This is the link to the empty resource:

<a href="http://www.1and1.com/?k_id=10219574" target="_blank">
    <en-media style="margin: 1em;" width="140" height="28" alt="Banner" border="0" align="right" type="text/html" hash="d41d8cd98f00b204e9800998ecf8427e"/>
</a>

The resource's type is text/html, maybe an iframe for an ad banner? Probably we can just ignore it.

@ylian
Copy link
Member

ylian commented Nov 23, 2015

Haha, the original webpage hasn't changed since 2007. The empty resource is a banner that doesn't load on the original webpage. Go to http://javascript.crockford.com/ and scroll to the bottom.

screen shot 2015-11-22 at 8 28 49 pm

@crispinb
Copy link
Author

Well it's not like much javascript-related has changed since 2007, has it? So Mr. Crockford can safely relax.

That build works well for me -- just imported the whole of the notebook that note came from without problems. It's an archive notebook with about 5000 pretty crufty old notes (lots of web clippings) so I thought it would give Quiver a good workout (up to now I've mostly used it for manually-typed notes).

Anyway thanks, that works for me.

@ylian
Copy link
Member

ylian commented Nov 23, 2015

Awesome. Thanks for the bug report.

@ylian
Copy link
Member

ylian commented Feb 9, 2016

Fixed in 3.0.1.

@ylian ylian closed this as completed Feb 9, 2016
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

2 participants