[chirp_users] [CHIRP] #97: CSV export writes chirp native XML file if filename doesn't end in .csv.
#97: CSV export writes chirp native XML file if filename doesn't end in .csv. -------------------------------+-------------------------------------------- Reporter: rct+chirp@… | Type: Bug Status: new | Priority: Stability Milestone: | Component: Memory Editor Version: 0.1.11 | Keywords: Platform: Linux | -------------------------------+-------------------------------------------- Export CSV only seems to write a CSV IFF the filename ends in .csv, otherwise it writes out a chirp native XML file.
Note: If the suggested filename + extension gets cleared in the process of navigating to the desired directory, no extension gets set. This might be the desired behavior, but it leads to this behavior where it looks like you should be saving a .csv file and instead wind up saving an xml file.
Note 2: the comparison is case sensitive too, so saving to radio.CSV gives an XML file as well.
chirp 0.1.11 freshly pulled via hg 4/30/2011. hg summary parent: 915:6ffa431cdbdd tip Make banks import and export properly branch: default commit: (clean) update: (current)
#97: CSV export writes chirp native XML file if filename doesn't end in .csv. --------------------------------+------------------------------------------- Reporter: rct+chirp@… | Owner: Type: Bug | Status: closed Priority: Stability | Milestone: Component: Memory Editor | Version: 0.1.11 Resolution: Fixed | Keywords: Platform: Linux | --------------------------------+------------------------------------------- Changes (by kk7ds):
* status: new => closed * resolution: => Fixed
Comment:
Fixed in changeset:919.
I opted to throw an error if an extension gets mangled or removed (case issues notwithstanding).
#97: CSV export writes chirp native XML file if filename doesn't end in .csv. --------------------------------+------------------------------------------- Reporter: rct+chirp@… | Owner: Type: Bug | Status: closed Priority: Stability | Milestone: Component: Memory Editor | Version: 0.1.11 Resolution: Fixed | Keywords: Platform: Linux | --------------------------------+-------------------------------------------
Comment (by rct+chirp@…):
While throwing an error is certainly better than writing the wrong file type, it seems to be a bit odd to throw an error based on what's typed in the file name part of the dialog box when I've already selected what type of file export I want (from the menu), and there is a file type selection in the dialog box (at least the linux gtk file dialog box) that says csv Files (*.csv).
#97: CSV export writes chirp native XML file if filename doesn't end in .csv. --------------------------------+------------------------------------------- Reporter: rct+chirp@… | Owner: Type: Bug | Status: closed Priority: Stability | Milestone: Component: Memory Editor | Version: 0.1.11 Resolution: Fixed | Keywords: Platform: Linux | --------------------------------+-------------------------------------------
Comment (by kk7ds):
The reason for that, at the moment, is that changing that filter drop-down doesn't auto-substitute the proper extension on the user's filename. I'm trying to figure out how to do that right now. If I can get that to behave properly, then I will avoid the error dialog and clamp to whatever they have selected in that box. Sound okay?
#97: CSV export writes chirp native XML file if filename doesn't end in .csv. --------------------------------+------------------------------------------- Reporter: rct+chirp@… | Owner: Type: Bug | Status: closed Priority: Stability | Milestone: Component: Memory Editor | Version: 0.1.11 Resolution: Fixed | Keywords: Platform: Linux | --------------------------------+-------------------------------------------
Comment (by rct+chirp@…):
Yes. I think that will behave a bit more like I would expect.
#97: CSV export writes chirp native XML file if filename doesn't end in .csv. --------------------------------+------------------------------------------- Reporter: rct+chirp@… | Owner: Type: Bug | Status: closed Priority: Stability | Milestone: Component: Memory Editor | Version: 0.1.11 Resolution: Fixed | Keywords: Platform: Linux | --------------------------------+-------------------------------------------
Comment (by kk7ds):
changeset:922
participants (1)
-
CHIRP Tracker