Skip to content

Handle additional types in ztypeName - #328

Merged
ktarbet merged 2 commits into
HydrologicEngineeringCenter:mainfrom
gyanz:patch-ztypeName
Nov 3, 2025
Merged

Handle additional types in ztypeName#328
ktarbet merged 2 commits into
HydrologicEngineeringCenter:mainfrom
gyanz:patch-ztypeName

Conversation

@gyanz

@gyanz gyanz commented Nov 1, 2025

Copy link
Copy Markdown
Contributor

The zTypeName function currently returns DATA_TYPE_UNDEFINED (abbreviated as "UDF") for grid, image, and file record types. This patch adds proper handling for these record types.

@sonarqubecloud

sonarqubecloud Bot commented Nov 1, 2025

Copy link
Copy Markdown

@ktarbet
ktarbet merged commit 2724d2d into HydrologicEngineeringCenter:main Nov 3, 2025
5 checks passed
@ktarbet

ktarbet commented Nov 3, 2025

Copy link
Copy Markdown
Contributor

@gyanz

note: The main branch of DSS now is only for DSS Version 7 support. You may wish to put this same change into the master branch if you need version 6 support.

@gyanz

gyanz commented Nov 6, 2025

Copy link
Copy Markdown
Contributor Author

@ktarbet What’s the standard way to apply the same change to two branches? Do I need to fork the repo and make the changes separately on each branch?

ktarbet pushed a commit that referenced this pull request Nov 6, 2025
* Add abbreviations for few data types

* Add handling for additional data types in ztypeName
@ktarbet

ktarbet commented Nov 6, 2025

Copy link
Copy Markdown
Contributor

Cherry pick like this: (I did this so I could practice/learn).

git clone https://github.com/HydrologicEngineeringCenter/hec-dss.git

cd hec-dss

git checkout -b cherry-pick-type-names

git cherry-pick 2724d2d


git push --set-upstream origin cherry-pick-type-names

PR is here: #332

ktarbet added a commit that referenced this pull request Nov 6, 2025
* Add abbreviations for few data types

* Add handling for additional data types in ztypeName

Co-authored-by: Gyan Basyal <gyanBasyalz@gmail.com>

note: This is a courtesy (learning) cherry pick to master branch (DSS Version 6 enabled.)   

Support for this branch (master) will need to be community support in the future.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants