Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

Pushing a property multiple times only pushed it once

Open
  • Is something wrong?
#59 0 comments 0 reactions 1 assignee View on GitHub

@adecler is already working on this.

Since Mar 5, 2024.

Assessment

This issue has not been assessed yet.

Description

type:bug
Description:

For something like this:

Config = new ExcelPushConfig
{
    Worksheet = "Elements",
    GoDeepInProperties = true,
    StartingCell = new CellAddress { Column = "A", Row = 3 },
    IncludePropertyNames = false,
    ObjectProperties = new List<string> {
        "Element.Name", "Element.ElementGroup", "Element.Type", "Element.Name",
        "Factors.A1toA3", "Factors.A4", "Factors.A5w"
    }
}

Element.Name is only added once instead of two. This results in the columns after teh second Element.Name being shifted to the left by one.

Steps to reproduce:

Push objects while using teh same property multiple times like above

Expected behaviour:

It should be possible to push the same property multiple times

Test file(s):

StructureEmbodiedCarbonTest.zip

Dominant language
C#
Stars
0
Forks
3
Avg merge
3d 22h
Merged PRs (30d)
3

Getting set up

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.

More from BHoM/Excel_Toolkit

All issues in BHoM/Excel_Toolkit

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.