Skip to content

GetMaterialCommand and GetTypologyCommand filter should be case insensitive #339

@jamesramsden-bh

Description

@jamesramsden-bh

The nature of the Materials library means that having to search for 'concrete' means having to run the search twice, for 'concrete' and 'Concrete' to find all relevant materials. I recommend that the filter is made case insensitive to allow such searches to find all relevant results with a single command.

e.g. line 85 of GetMaterialCommand could be

return materialObjects.Where(m => (m as IEnergyMaterialOpaque).Name.ToLower().Contains(command.Filter.ToLower())).ToList();

Please discuss if there are any use cases where case sensitivity is necessary.

Metadata

Metadata

Assignees

No one assigned

    Labels

    type:featureNew capability or enhancement
    No fields configured for Feature.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions