darknessomi / darknessomi/excellibrary

Patch for /trunk/src/ExcelLibrary/DataSetHelper.cs

Open
#110 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

auto-migrated Type-Patch
Dominant language
C#
Stars
22
Forks
8
PR merge metrics
No merged PRs in 30d

Description

In general, excel names are case insensitive. Worksheet names should be case 
insensitive

Original issue reported on code.google.com by m.nieto....@gmail.com on 20 Sep 2011 at 9:12

Attachments:

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Review /trunk/src/ExcelLibrary/DataSetHelper.cs and compare it with the attached DataSetHelper.cs.patch. Trace how worksheet names are handled, then verify that names differing only by letter case resolve as the same worksheet while existing worksheet behavior remains intact.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
50/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.