JDownloader Community - Appwork GmbH
 

Reply
 
Thread Tools Display Modes
  #161  
Old 24.07.2023, 13:35
pspzockerscene's Avatar
pspzockerscene pspzockerscene is offline
Community Manager
 
Join Date: Mar 2009
Location: Deutschland
Posts: 73,118
Default

Quote:
Originally Posted by CheatBaker View Post
Actually, I didn't see other case, but il do not means it can't occur.
I thought : Maybe if you put [A-Z] in your regular expression, you will not have any trouble with this kind of problem, except if it causes unwanted effects.
Done.

Quote:
Originally Posted by CheatBaker View Post
If I dared, I would suggest you some other improvements :
In a file name, when you encounter a character forbidden, especially by MS-DOS, you replace it by an underscore one. It is clever.
But some of them are useful to explain and light the structure and the meaning of this file name. So I have to replace them manually by other acceptable characters. My correspondances are the following :
Agreed and done:
We already got a function for this but we are not yet using it globally.
Therefore, I've updated the arte.tv plugin to make use of it.
You can see our current character replacements here in our source code:
github.com/mirror/jdownloader/blob/master/src/jd/plugins/Plugin.java#L779

For all code changes mentioned in this post, the following information applies:
Wartest du auf einen angekündigten Bugfix oder ein neues Feature?
Updates werden nicht immer sofort bereitgestellt!
Bitte lies unser Update FAQ! | Please read our Update FAQ!

---
Are you waiting for recently announced changes to get released?
Updates to not necessarily get released immediately!
Bitte lies unser Update FAQ! | Please read our Update FAQ!


-psp-
__________________
JD Supporter, Plugin Dev. & Community Manager

Erste Schritte & Tutorials || JDownloader 2 Setup Download
Spoiler:

A users' JD crashes and the first thing to ask is:
Quote:
Originally Posted by Jiaz View Post
Do you have Nero installed?
Reply With Quote
  #162  
Old 24.07.2023, 15:33
pspzockerscene's Avatar
pspzockerscene pspzockerscene is offline
Community Manager
 
Join Date: Mar 2009
Location: Deutschland
Posts: 73,118
Default

Additional information:
We got a ticket regarding this topic:


Once that is done, we will provide a default set of replacements which can be user-defined.
The "replace invalid to underscore" functionality will remain but only after the user-defined replacements are done so mostly you shouldn't "loose" any information anymore then.
Once the mentioned ticket is done, I'll reply in this thread one more time.
__________________
JD Supporter, Plugin Dev. & Community Manager

Erste Schritte & Tutorials || JDownloader 2 Setup Download
Spoiler:

A users' JD crashes and the first thing to ask is:
Quote:
Originally Posted by Jiaz View Post
Do you have Nero installed?
Reply With Quote
  #163  
Old 25.07.2023, 18:40
CheatBaker CheatBaker is offline
Super Loader
 
Join Date: May 2022
Posts: 28
Smile file characters replacement

Quote:
Originally Posted by pspzockerscene View Post
Additional information:
Controlling: customized(advanced settings) filename characters replacer
Status: Assigned Created:
2017-06-16T07:58:49Z
Priority: Normal Progress: 0%
Last Update:
2023-07-25T11:17:59Z
Details: here
Hello,
I saw that you have changed the way of handle filenames. Thanks. I did guess that you used the old way (before the great revolution of Arte, last year). It is better. Maybe you can apply that to the package name too, even if it doesn't matter for downloadings, just esthetic.
I'm very interested by your user-defined replacement. It will better more. Thanks too for that.

About the quote : If I try to click on "here" after "details", I obtain a message wich ask me to identify me and it doesn't want me to give my log Id . I suppose I'm not granted .

About github.com/mirror/jdownloader/blob/master/src/jd/plugins/Plugin.java#L779 :
I saw in this code the characters changes, except the double quote (") which must be treated in another function, but I don't understand some details :
- The "!" is converted in "¡" and that is not useful because MS-DOS accepts it. In spite of the instruction is flagged "not illegal" two lines upper, it is conserved. Maybe the need of this conversion is elsewhere, or is it a forgetting ?
- Maybe the slash (\) could be replaced by "∖"(Ascii=92) like double slash (\\), instead of "'" (single quote) which is not very logical.
Otherwise, it seems correct (except the colon. I prefer minus, but it is my own convenience, and I can't impose it to everybody)

Your user-defined adding will solve it, I presume.

Last minute :
ARTE_NEXT is back. I don't understand why this platform appears again. It gives to me twice more work for creating good filenames. Happily, on the package name, it doesn't, and that help me for correcting. I hope that it is temporary.

Regards
CB

Last edited by CheatBaker; 26.07.2023 at 00:42. Reason: Last minute
Reply With Quote
  #164  
Old 26.07.2023, 15:07
pspzockerscene's Avatar
pspzockerscene pspzockerscene is offline
Community Manager
 
Join Date: Mar 2009
Location: Deutschland
Posts: 73,118
Default

Quote:
Originally Posted by CheatBaker View Post
Maybe you can apply that to the package name too, even if it doesn't matter for downloadings, just esthetic
Done.

Quote:
Originally Posted by CheatBaker View Post
About the quote : If I try to click on "here" after "details", I obtain a message wich ask me to identify me and it doesn't want me to give my log Id . I suppose I'm not granted .
The details view is only for internal usage / developers.

Quote:
Originally Posted by CheatBaker View Post
I saw in this code the characters changes, except the double quote (") which must be treated in another function, but I don't understand some details
Double-quote is changed to single quote, see:
github.com/mirror/jdownloader/blob/5f50ef81a535c64d31713ed043587eac7a3da057/src/jd/plugins/Plugin.java#L793C22-L793C22

Quote:
Originally Posted by CheatBaker View Post
- The "!" is converted in "¡" and that is not useful because MS-DOS accepts it. In spite of the instruction is flagged "not illegal" two lines upper, it is conserved. Maybe the need of this conversion is elsewhere, or is it a forgetting ?
That line is commented out - "!" is not changed as it is allowed in filenames.
Your English is hard to understand. Maybe you want to answer in your native language and/or use Google translage (sorry - I'm not meaning this in any bad way).

Quote:
Originally Posted by CheatBaker View Post
- Maybe the slash (\) could be replaced by "∖"(Ascii=92) like double slash (\\), instead of "'" (single quote) which is not very logical.
Looks like you mis-read that one too. Backslash is also already replaced by "∖" and slash by "⁄".

Quote:
Originally Posted by CheatBaker View Post
Otherwise, it seems correct (except the colon. I prefer minus, but it is my own convenience, and I can't impose it to everybody)

Your user-defined adding will solve it, I presume.
Yes, you will be able to customize this in the future.

Quote:
Originally Posted by CheatBaker View Post
Last minute :
ARTE_NEXT is back
I'm not an ARTE user so I have no idea what "ARTE_NEXT" is.

Fo all code changes mentioned in this post, the following information applies:
Bitte auf das nächste CORE-Update warten!

Please wait for the next CORE-Update!

Wartest du auf einen angekündigten Bugfix oder ein neues Feature?
Updates werden nicht immer sofort bereitgestellt!
Bitte lies unser Update FAQ! | Please read our Update FAQ!

---
Are you waiting for recently announced changes to get released?
Updates to not necessarily get released immediately!
Bitte lies unser Update FAQ! | Please read our Update FAQ!


-psp-
__________________
JD Supporter, Plugin Dev. & Community Manager

Erste Schritte & Tutorials || JDownloader 2 Setup Download
Spoiler:

A users' JD crashes and the first thing to ask is:
Quote:
Originally Posted by Jiaz View Post
Do you have Nero installed?
Reply With Quote
  #165  
Old 08.08.2023, 00:25
CheatBaker CheatBaker is offline
Super Loader
 
Join Date: May 2022
Posts: 28
Smile Acknowledgment of receipt

Quote:
Originally Posted by pspzockerscene View Post
Your English is hard to understand. Maybe you want to answer in your native language and/or use Google translage (sorry - I'm not meaning this in any bad way).
I apologize for this delay in responding and also for my abominable English. I'm French, and we're not known to be good at languages. All the text is now translated by Google as you suggested to me. Hope you can understand it better.

Quote:
Originally Posted by pspzockerscene View Post
Done.
Ok, Thank You

Quote:
Originally Posted by pspzockerscene View Post
The details view is only for internal usage / developers.
OK, it's noted


Quote:
Originally Posted by pspzockerscene View Post
Double-quote is changed to single quote, see:
github.com/mirror/jdownloader/blob/5f50ef81a535c64d31713ed043587eac7a3da057/src/jd/plugins/Plugin.java#L793C22-L793C22

Looks like you mis-read that one too. Backslash is also already replaced by "" and slash by "/".
It's Ok. I didn't quite understand these instructions. Now, I do.


Quote:
Originally Posted by pspzockerscene View Post
That line is commented out - "!" is not changed as it is allowed in filenames.
In fact, the "!" always becomes "¡" as you can see on this url:
**External links are only visible to Support Staff****External links are only visible to Support Staff**



Quote:
Originally Posted by pspzockerscene View Post
Yes, you will be able to customize this in the future.
Understood, I can't wait.

Quote:
Originally Posted by pspzockerscene View Post
I'm not an ARTE user so I have no idea what "ARTE_NEXT" is.
It doesn't matter anymore: I found a setting that gives me the right result.

**********
Another problem
By the way, this url is not loading by JD
**External links are only visible to Support Staff****External links are only visible to Support Staff**

Thank you for your attention.
Reply With Quote
  #166  
Old 08.08.2023, 13:32
pspzockerscene's Avatar
pspzockerscene pspzockerscene is offline
Community Manager
 
Join Date: Mar 2009
Location: Deutschland
Posts: 73,118
Default

Regarding the character replacement setting:
I will post in this thread once we got news about that.
Until then, you can use external programs to e.g. revert replacements or change them. That shouldn't be hard to setup.

Regarding your last link:
That also fails here in browser.
Screenshot (open SPOILER to see):
Spoiler:

-> You might need to contact the ARTE support about this one.
__________________
JD Supporter, Plugin Dev. & Community Manager

Erste Schritte & Tutorials || JDownloader 2 Setup Download
Spoiler:

A users' JD crashes and the first thing to ask is:
Quote:
Originally Posted by Jiaz View Post
Do you have Nero installed?
Reply With Quote
  #167  
Old 14.06.2024, 17:13
pspzockerscene's Avatar
pspzockerscene pspzockerscene is offline
Community Manager
 
Join Date: Mar 2009
Location: Deutschland
Posts: 73,118
Default

Character replacer changes are live, see:
https://board.jdownloader.org/showthread.php?t=95411
__________________
JD Supporter, Plugin Dev. & Community Manager

Erste Schritte & Tutorials || JDownloader 2 Setup Download
Spoiler:

A users' JD crashes and the first thing to ask is:
Quote:
Originally Posted by Jiaz View Post
Do you have Nero installed?
Reply With Quote
Reply

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump

All times are GMT +2. The time now is 01:03.
Provided By AppWork GmbH | Privacy | Imprint
Parts of the Design are used from Kirsch designed by Andrew & Austin
Powered by vBulletin® Version 3.8.10 Beta 1
Copyright ©2000 - 2024, Jelsoft Enterprises Ltd.