kiwix / kiwix-xulrunner

[ARCHIVED] Legacy Kiwix desktop solution for Windows/macOS/Linux
https://download.kiwix.org/release/kiwix-xulrunner/
GNU General Public License v3.0
111 stars 28 forks source link

Make it impossible to download twice a content on Android #403

Closed kelson42 closed 7 years ago

kelson42 commented 7 years ago

Steps:

What we get:

What we want:

mhutti1 commented 7 years ago

@kelson42 I am struggling to reproduce this.

EladKeyshawn commented 7 years ago

@mhutti1 Are you on this ? I don't want to work twice 😉

mhutti1 commented 7 years ago

@EladKeyshawn I can't reproduce it so Im not working on it.

EladKeyshawn commented 7 years ago

@mhutti1 really ? this is actually a bug that should occur on every device.

mhutti1 commented 7 years ago

@EladKeyshawn Yeah I fixed it on my device and the emulator I have. So I thought I fixed it fully,

EladKeyshawn commented 7 years ago

@mhutti1 No need I fixed it. now pushing

kelson42 commented 7 years ago

@EladKeyshawn Seems to perfectly works now :)

kelson42 commented 7 years ago

@EladKeyshawn @mhutti1 This bug has re-appeared.

EladKeyshawn commented 7 years ago

Can you give me a specific Zim file with which it happens?

On Mon, Jan 2, 2017, 12:20 PM Kelson notifications@github.com wrote:

Reopened #403 https://github.com/kiwix/kiwix/issues/403.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kiwix/kiwix/issues/403#event-908899905, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVJuYHIdGcZlvFs5nynti1WlVke3tcrks5rOM9ogaJpZM4LW3U8 .

kelson42 commented 7 years ago

@EladKeyshawn I mostly test with both Wikimed (English) files

EladKeyshawn commented 7 years ago

Generally even without my patch the downloading files should no longer appear in the first place. So we I think it has to do with the most unique parameter separating those files. I took the full name of the Zim file. Maybe there are duplicates?

On Mon, Jan 2, 2017, 12:31 PM Kelson notifications@github.com wrote:

@EladKeyshawn https://github.com/EladKeyshawn I mostly test with both Wikimed (English) files

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kiwix/kiwix/issues/403#issuecomment-269955129, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVJuRmMJ8Bzmzp8Ld1AJUSLn26LFNRXks5rONHmgaJpZM4LW3U8 .

kelson42 commented 7 years ago

@EladKeyshawn ZIM file should be identified by there UID, not by there title or filename

EladKeyshawn commented 7 years ago

Yes but I'm not sure we save that in the DB I need to check

On Mon, Jan 2, 2017, 1:04 PM Kelson notifications@github.com wrote:

@EladKeyshawn https://github.com/EladKeyshawn ZIM file should be identified by there UID, not by there title or filename

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kiwix/kiwix/issues/403#issuecomment-269958552, or mute the thread https://github.com/notifications/unsubscribe-auth/AQVJuYdyfs3hTcV7kb8MUBPc-JYjDrnxks5rONnTgaJpZM4LW3U8 .

mhutti1 commented 7 years ago

@kelson42 Can you give a more specific reproduction case for this?

kelson42 commented 7 years ago

@mhutti1 @EladKeyshawn If I try to download quickly the two Wikimed (in English) ZIM files, only one disappear from the "online" tab. Even by trying to download the "bollywood" file, it often does not disappear from the "online" tab.

mhutti1 commented 7 years ago

@kelson42 How about now?

kelson42 commented 7 years ago

@mhutti1 Looks good now :)