Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Correct 5ded6212-e8d4-4206-9025-cb5991bd2f80.md (Insert text into cell) #20

Merged
merged 2 commits into from
Mar 11, 2016

Conversation

CCAchris
Copy link
Contributor

@CCAchris CCAchris commented Feb 6, 2016

Added an extra line of code to both the c# and vb sections to correct a bug that occurs when inserting a new cell with a column name greater than Z. The existing code mistakenly inserts cell AA1 in front of B1. The addition of the string length comparison ensures that the column string lengths are the same before sorting the strings.

This addition adds a statement comparing the column string lengths to determine if they are the same before sorting the strings. Without the addition, the String.Compare will mistakenly insert cell AA1 before B1 leading to a corrupted file.
@msftclas
Copy link

msftclas commented Feb 6, 2016

Hi @CCAchris, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

laschultz added a commit that referenced this pull request Mar 11, 2016
Correct 5ded6212-e8d4-4206-9025-cb5991bd2f80.md (Insert text into cell)
@laschultz laschultz merged commit 5cb4da2 into OfficeDev:master Mar 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants