Skip opening holding detail dialog for cash positions

libs/ui/src/lib/top-holdings/top-holdings.component.html

@@ -121,13 +121,7 @@
                 *matRowDef="let row; columns: displayedColumns"
                 mat-row
                 [class.cursor-pointer]="canShowDetails(row)"
-                (click)="
+                (click)="onClickHolding(row)"
-                  canShowDetails(row) &&
-                    onClickHolding({
-                      dataSource: row.position.assetProfile.dataSource,
-                      symbol: row.position.assetProfile.symbol
-                    })
-                "
               ></tr>
               <tr
                 *matFooterRowDef="displayedColumns"

libs/ui/src/lib/top-holdings/top-holdings.component.ts

@@ -87,8 +87,15 @@ export class GfTopHoldingsComponent implements OnChanges {
     return !!holding?.position && canOpenHoldingDetail(holding.position);
   }
 
-  public onClickHolding({ dataSource, symbol }: AssetProfileIdentifier) {
+  public onClickHolding({ position }: { position?: PortfolioPosition }) {
-    this.holdingClicked.emit({ dataSource, symbol });
+    if (!position || !canOpenHoldingDetail(position)) {
+      return;
+    }
+
+    this.holdingClicked.emit({
+      dataSource: position.assetProfile.dataSource,
+      symbol: position.assetProfile.symbol
+    });
   }
 
   public onShowAllHoldings() {