[GUI Plugin] Quality Check - Page 13

kiwidude · 05-17-2011, 03:54 PM

Changes in this release:

Prevent ePubs with encrypted fonts from showing as being books with DRM
Rename Abort to Cancel on the progress dialog

drMerry · 05-23-2011, 04:05 PM

Just a new check idea:
Duplicate sub-genres

To test books that have for example both these genres: Fiction, Fiction.Science
Maybe you could also test Fiction.Science, Fiction.Space but I think this is not very useful the first would be a great option

I only use partitioning in tags, but since you can set this manually, it would be useful to have a setting for selecting which parameters to test.

kiwidude · 05-25-2011, 08:59 AM

Changes in this release:

Add ePub check for invalid namespaces
Add a viewable log for the checks to allow user to see details of errors, missing files etc
Add a Help file describing all of the checks and how to solve them

drMerry · 05-28-2011, 04:29 AM

I can not use custom shortcuts anymore.
Reported it on bugs: bug 789481

drMerry · 05-28-2011, 05:39 AM

Your log is great!
This made me think of a new option:
Check epub with specified file-types.
For example, I mentioned I had epubs with .exe files in it (a web-page I added that calibre combined) and a lot of files with thumbs.db

It would be great to see this kind of files (possible custom added extensions of paterns?)

And anonther option: Epub containing files lager than ....
This would give the option to see the difference of a file with 300 small html files and a file with 2 html files and a very large cover.

kiwidude · 05-28-2011, 05:51 AM

Quote:

Originally Posted by drMerry

I can not use custom shortcuts anymore.
Reported it on bugs: bug 789481

Please STOP spamming all the plugin threads with this message. Once was enough. This plugin doesn't even have shortcuts.

drMerry · 05-28-2011, 06:40 AM

Quote:

Originally Posted by kiwidude

Please STOP spamming all the plugin threads with this message. Once was enough. This plugin doesn't even have shortcuts.

, sorry. I wanted to post it on duplicate and open with, this one was en error.
I wanted to post it on the plugins that use a shortcut so not every-one using one of the plugins could see it in the post of their plugin.

Was not ment to 'flood' the forum

kiwidude · 05-28-2011, 07:22 AM

Ok, I have deleted your other two posts. Even posting on two threads is one too many. Why would I want to have to deal with two discussions on the same topic, and confused users having to look at multiple threads for repeated answers?

That you say you had this issue after upgrading Calibre and not after upgrading specific plugins indicates that it is a generic issue. You raised a ticket which is the right thing to do in this situation. I have responded on your ticket - which is to say that I have no such problem with shortcuts with the latest release, they are all working fine on my plugins. Any further discussion on this should remain on that ticket.

drMerry · 05-28-2011, 10:57 AM

Removed post.
This is no production so I will put it in develop section

Perkin · 06-05-2011, 07:13 AM

Kiwidude, Another check which may be useful for epubs is to check for large html filesizes which cause errors on some readers.

Will then easily allow the user to do a reconvert with a better split size

kiwidude · 06-05-2011, 11:43 AM

Quote:

Originally Posted by Perkin

Kiwidude, Another check which may be useful for epubs is to check for large html filesizes which cause errors on some readers.

Will then easily allow the user to do a reconvert with a better split size

@Perkin - that sounds a good idea. Do you know that the threshold value is that I should be checking for?

kovidgoyal · 06-05-2011, 12:01 PM

260KB

theducks · 06-05-2011, 12:08 PM

Outstanding idea

theducks · 06-05-2011, 12:41 PM

I just ran the check for extra files.
books from Feedbooks showed up
they have this usage in the stylesheet

Code:

@import "page.css";

So is the usage correct or the check wrong?
Note once I open the file in Sigil, I have to correct the entry to read:

Code:

@import "../Styles/page.css";

.
Sigil 3.4 flags the file as 'unused' (IIRC Sigil does not check any of the contents of the Stylesheet(s) )

kiwidude · 06-05-2011, 12:52 PM

@theducks - what my check does is look for whether there are files in the ePub that are not listed in the .opf manifest. It doesn't look in the html files or stylesheet files at all.

So my assumption is that any file that is intended is part of your ePub should be listed in the manifest.

It would appear that whoever has authored those books from Feedbooks has decided that they do not consider it relevant to include their stylesheet in the manifest. I don't know the ePub specification to know whether this is "valid" or not. I know Valloric watches the spec pretty closely, so if Sigil treats these as "unused" it sounds like they should be. Or it could just be a bug, Sigil has plenty of those.

I know that ePubs will "work" without having files listed in the manifest, but of course this plugin is about identifying "less than ideal" scenarios. If in fact it is seen as an acceptable convention to not include .css files in the manifest then perhaps I have to make a special case exemption. Alternatively I could change finding such situations into a "warning" and just displaying in the log.

I welcome anyone's input on how these should be treated, if any differently than currently.

05-17-2011, 03:54 PM	#181
kiwidude Calibre Plugins Developer Posts: 4,636 Karma: 2162064 Join Date: Oct 2010 Location: Australia Device: Kindle Oasis	v1.5.2 Released Changes in this release: Prevent ePubs with encrypted fonts from showing as being books with DRM Rename Abort to Cancel on the progress dialog

05-25-2011, 08:59 AM	#183
kiwidude Calibre Plugins Developer Posts: 4,636 Karma: 2162064 Join Date: Oct 2010 Location: Australia Device: Kindle Oasis	v1.5.3 Released Changes in this release: Add ePub check for invalid namespaces Add a viewable log for the checks to allow user to see details of errors, missing files etc Add a Help file describing all of the checks and how to solve them

05-28-2011, 10:57 AM	#189
drMerry Addict Posts: 293 Karma: 21022 Join Date: Mar 2011 Location: NL Device: Sony PRS-650	Removed post. This is no production so I will put it in develop section Last edited by drMerry; 05-28-2011 at 11:18 AM.

06-05-2011, 12:41 PM	#194
theducks Well trained by Cats Posts: 29,799 Karma: 54830978 Join Date: Aug 2009 Location: The Central Coast of California Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A	I just ran the check for extra files. books from Feedbooks showed up they have this usage in the stylesheet Code: @import "page.css"; So is the usage correct or the check wrong? Note once I open the file in Sigil, I have to correct the entry to read: Code: @import "../Styles/page.css"; . Sigil 3.4 flags the file as 'unused' (IIRC Sigil does not check any of the contents of the Stylesheet(s) )

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
[GUI Plugin] Clipboard Search	kiwidude	Plugins	29	04-02-2024 10:05 PM
[GUI Plugin] Search the Internet	kiwidude	Plugins	433	04-01-2024 05:48 PM
[GUI Plugin] Open With	kiwidude	Plugins	403	04-01-2024 08:39 AM
[GUI Plugin] Kindle Collections (old)	meme	Plugins	2070	08-11-2014 12:02 AM
[GUI Plugin] Book Sync Deprecated	kiwidude	Plugins	111	06-07-2011 07:47 PM

05-23-2011, 04:05 PM	#182
drMerry Addict Posts: 293 Karma: 21022 Join Date: Mar 2011 Location: NL Device: Sony PRS-650	Just a new check idea: Duplicate sub-genres To test books that have for example both these genres: Fiction, Fiction.Science Maybe you could also test Fiction.Science, Fiction.Space but I think this is not very useful the first would be a great option I only use partitioning in tags, but since you can set this manually, it would be useful to have a setting for selecting which parameters to test.

05-28-2011, 04:29 AM	#184
drMerry Addict Posts: 293 Karma: 21022 Join Date: Mar 2011 Location: NL Device: Sony PRS-650	I can not use custom shortcuts anymore. Reported it on bugs: bug 789481

05-28-2011, 05:39 AM	#185
drMerry Addict Posts: 293 Karma: 21022 Join Date: Mar 2011 Location: NL Device: Sony PRS-650	Your log is great! This made me think of a new option: Check epub with specified file-types. For example, I mentioned I had epubs with .exe files in it (a web-page I added that calibre combined) and a lot of files with thumbs.db It would be great to see this kind of files (possible custom added extensions of paterns?) And anonther option: Epub containing files lager than .... This would give the option to see the difference of a file with 300 small html files and a file with 2 html files and a very large cover.

05-28-2011, 07:22 AM	#188
kiwidude Calibre Plugins Developer Posts: 4,636 Karma: 2162064 Join Date: Oct 2010 Location: Australia Device: Kindle Oasis	Ok, I have deleted your other two posts. Even posting on two threads is one too many. Why would I want to have to deal with two discussions on the same topic, and confused users having to look at multiple threads for repeated answers? That you say you had this issue after upgrading Calibre and not after upgrading specific plugins indicates that it is a generic issue. You raised a ticket which is the right thing to do in this situation. I have responded on your ticket - which is to say that I have no such problem with shortcuts with the latest release, they are all working fine on my plugins. Any further discussion on this should remain on that ticket.

06-05-2011, 07:13 AM	#190
Perkin Guru Posts: 655 Karma: 64171 Join Date: Sep 2010 Location: Kent, England, Sol 3, ZZ9 plural Z Alpha Device: Sony PRS-300, Kobo Aura HD, iPad (Marvin)	Kiwidude, Another check which may be useful for epubs is to check for large html filesizes which cause errors on some readers. Will then easily allow the user to do a reconvert with a better split size

06-05-2011, 12:01 PM	#192
kovidgoyal creator of calibre Posts: 43,851 Karma: 22666666 Join Date: Oct 2006 Location: Mumbai, India Device: Various	260KB

06-05-2011, 12:08 PM	#193
theducks Well trained by Cats Posts: 29,799 Karma: 54830978 Join Date: Aug 2009 Location: The Central Coast of California Device: Kobo Libra2,Kobo Aura2v1, K4NT(Fixed: New Bat.), Galaxy Tab A	Outstanding idea

06-05-2011, 12:52 PM	#195
kiwidude Calibre Plugins Developer Posts: 4,636 Karma: 2162064 Join Date: Oct 2010 Location: Australia Device: Kindle Oasis	@theducks - what my check does is look for whether there are files in the ePub that are not listed in the .opf manifest. It doesn't look in the html files or stylesheet files at all. So my assumption is that any file that is intended is part of your ePub should be listed in the manifest. It would appear that whoever has authored those books from Feedbooks has decided that they do not consider it relevant to include their stylesheet in the manifest. I don't know the ePub specification to know whether this is "valid" or not. I know Valloric watches the spec pretty closely, so if Sigil treats these as "unused" it sounds like they should be. Or it could just be a bug, Sigil has plenty of those. I know that ePubs will "work" without having files listed in the manifest, but of course this plugin is about identifying "less than ideal" scenarios. If in fact it is seen as an acceptable convention to not include .css files in the manifest then perhaps I have to make a special case exemption. Alternatively I could change finding such situations into a "warning" and just displaying in the log. I welcome anyone's input on how these should be treated, if any differently than currently.

Advert

Advert