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

8.3 filename format does not supported trough wifi upload #315

Open
pagabee opened this issue Mar 4, 2022 · 6 comments
Open

8.3 filename format does not supported trough wifi upload #315

pagabee opened this issue Mar 4, 2022 · 6 comments

Comments

@pagabee
Copy link

pagabee commented Mar 4, 2022

Did you test the latest bugfix-2.0.x code?

Yes, and the problem still exists.

Bug Description

If you copy a gcode file trough wifi and if its name is 5 or more characters long the copied filename on SD card will be xxxxx~1.gco.
This is not a 8.3 filename format.

The checking of the lengt of the filename happens in the wifi_module.ccp
static bool longName2DosName(const char *longName, char *dosName)

if (i >= 5) {
  strcat_P(dosName, PSTR("~1.GCO"));
  return dosName[0] != '\0';
}

What is the reason for this?

Bug Timeline

Old bug

Expected behavior

8.3 filename support

Actual behavior

qwer.gco --> qwer.gco

qwert.gco --> qwert~1.gco

qwertz.gco --> qwert~1.gco

qwertzu.gco --> qwert~1.gco

qwertzui.gco --> qwert~1.gco

qwertzuio.gco --> qwert~1.gco

Steps to Reproduce

Please upload a file trough wifi to the SD card.
I'm currently using ESP3D 3.0 and tested it with the MksWifi.bin firmware.

Version of Marlin Firmware

02000902

Printer model

No response

Electronics

MKS Robin Nano V3

Add-ons

MKS TS35 Display + MKS WIFI modul

Bed Leveling

No response

Your Slicer

No response

Host Software

No response

Additional information & file uploads

IMG_3754

@pagabee
Copy link
Author

pagabee commented Mar 6, 2022

I modified the code:

if (i > 8) {
  dosName[6] = '\0';
  dosName[7] = '\0';
  dosName[8] = '\0';
  strcat_P(dosName, PSTR("~1.GCO"));
  return dosName[0] != '\0';
}

It works, but i don't know if it's correct.

@MKS-hosiey
Copy link

The character limit of the display interface, currently only so many characters can be displayed

@pagabee
Copy link
Author

pagabee commented Mar 11, 2022

I copied directly files with long name to the SD card.
The TS35 display shows maximum 12 characters after that comes the ~.

Files on SD card
IMG_3767
IMG_3768
IMG_3769
IMG_3770
IMG_3771

@pagabee
Copy link
Author

pagabee commented May 20, 2022 via email

@snow79
Copy link

snow79 commented Jun 1, 2022

Hello, I modified the code to this. Please search for the “~1.GCO” in the wifi_module.ccp file. There is link in the bug description. snow79 @.***> (időpont: 2022. máj. 19., Cs, 19:52) ezt írta:

I modified the code: if (i > 8) { dosName[6] = '\0'; dosName[7] = '\0'; dosName[8] = '\0'; strcat_P(dosName, PSTR("~1.GCO")); return dosName[0] != '\0'; } It works, but i don't know if it's correct. Where you have find this code??? — Reply to this email directly, view it on GitHub <#315 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATVGG2E2EB3X3RH54TEU2ITVKZ5ULANCNFSM5P5YLO4A . You are receiving this because you authored the thread.Message ID: </issues/315/1132012780@ github.com>

yes but... you have solved or not???
i try with your solution, but the problem presist... now it's not 5 char but 8 ....

@pagabee
Copy link
Author

pagabee commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants