The icontype parameter of KPI shortcode doesn't have any effect

[JRichmond27]

Description

Including a value for the icontype parameter for an item in the kpis shortcode does not seem to have any effect on the resulting HTML output. No matter what, the icon CSS class ends up starting with "fa".

Actual Behavior

Whether the icontype parameter is included or not, the resulting icon CSS class always starts with "fa".

Expected Behavior

If the icontype parameter is omitted, then the icon CSS class should start with "fa".

If the icontype parameter has a value specified, then the resulting icon CSS class should begin with the specified value.

Steps to Reproduce

1. Add an HTML block to any page
2. Edit the block and add this Lava to the content area:

{[ kpis ]}
    [[ kpi value:'1234' color:'orange' icontype:'xyz' icon:'icon-class' ]][[ endkpi ]] 
{[ endkpis ]} 

3. Save the block
4. Inspect the page's HTML and notice that the icon CSS class for the KPI is fa fa-fw icon-class instead of xyz fa-fw icon-class like it should be

Issue Confirmation

• Perform a search on the Github Issues to see if your bug or enhancement is already reported.
• Try to reproduce the problem on a fresh install or on the demo site.

Rock Version

16.3 & 16.5

Client Culture Setting

en-US