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

Importing an open .csv file causes application to crash #407

Closed
1 task done
adgiles opened this issue Jun 26, 2017 · 3 comments
Closed
1 task done

Importing an open .csv file causes application to crash #407

adgiles opened this issue Jun 26, 2017 · 3 comments

Comments

@adgiles
Copy link
Contributor

adgiles commented Jun 26, 2017

Tasks to be done June 6th 2018

  • Verify CMS issues addressed in CC and MT

Background:

Users at times need to be able to import and export files such as tables into ArcGIS Pro with the file already opened by another application.

Current State:
If you have a csv file open in excel and try to import the same file into ArcMap or ArcGIS Pro the application crashes.

Expected Behavior:
A user can import any file no matter if it is opened by another application or not. The applications to be aware of being opened by a user:

NotePad++
Notepad
Wordpad
Excel

@NatalieCampos NatalieCampos changed the title Trying to import a csv file that is open in another application crashes ArcMap Importing an open csv file causes application to crash Sep 7, 2017
@dfoll dfoll changed the title Importing an open csv file causes application to crash Importing an open .csv file causes application to crash Sep 7, 2017
@lfunkhouser lfunkhouser added A-bug and removed B - Bug labels Oct 6, 2017
@csmoore
Copy link
Member

csmoore commented Nov 14, 2017

I have not seen this happen with notepad, but definitely with Excel which locks the file - here is one (of many) workaround(s) when this is ready to work on:

        // Somewhat hack-ish - needed check for when file is locked or can't be opened
        // (e.g. because of open/being viewed somewhere else-like Excel)
        public bool IsFileOpenable(string filePath)
        {
            try
            {
                using (File.Open(filePath, FileMode.Open)) { }
            }
            catch (IOException ex)
            {
                System.Diagnostics.Trace.WriteLine("ERROR: " + " Message: " + ex.Message + 
                    " Could not open file : " + filePath);
                return false;
            }

            return true;
        }

@lfunkhouser lfunkhouser added this to the April Release - for CyberTech milestone Feb 14, 2018
saip added a commit to CTLocalGovTeam/coordinate-conversion-addin-dotnet that referenced this issue Mar 27, 2018
Esri#457 - Inputting coordinates manually do not populate the list function
@topowright-zz topowright-zz self-assigned this Apr 3, 2018
@topowright-zz
Copy link
Contributor

Verifed this has now been fixed in our version for ArcGIS Pro 2.1

@lfunkhouser
Copy link
Member

lfunkhouser commented Apr 17, 2018

@topowright This issue still has the task labels on it - This issue still requires a doc update and should not be "Done" http://solutions.arcgis.com/defense/help/coordinate-conversion/resources/release-notes.htm

Removing the "E-verified" label.

@lfunkhouser lfunkhouser added this to CT - Fixed in CT Project May 16, 2018
@lfunkhouser lfunkhouser added this to the JUN 2018 Sprint 8 milestone Jun 6, 2018
@lfunkhouser lfunkhouser moved this from CT - Fixed to To Verify in CT Project Jun 6, 2018
@topowright-zz topowright-zz self-assigned this Jun 12, 2018
@lfunkhouser lfunkhouser moved this from To Verify to Closed in CT Project Nov 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Development

No branches or pull requests

5 participants