Don't show users as unpaid if one of their payment attempts succeeded (…

…#10290)

* use has_paid when determining tooltip

* only show successful transaction for refund

* revert changes to payment_controller

app/webpacker/components/RegistrationsV2/RegistrationAdministration/AdministrationTableRow.jsx

@@ -21,15 +21,15 @@ import { countries } from '../../../lib/wca-data.js.erb';
 const truncateComment = (comment) => (comment?.length > 12 ? `${comment.slice(0, 12)}...` : comment);
 
 function RegistrationTime({
-  timestamp, registeredOn, paymentStatuses, paidOn, usesPaymentIntegration,
+  timestamp, registeredOn, paymentStatuses, hasPaid, paidOn, usesPaymentIntegration,
 }) {
   if (timestamp) {
     return getRegistrationTimestamp(paidOn ?? registeredOn);
   }
 
   const mostRecentPaymentStatus = paymentStatuses ? paymentStatuses[0] : 'unpaid';
 
-  if (usesPaymentIntegration && mostRecentPaymentStatus !== 'succeeded') {
+  if (usesPaymentIntegration && !hasPaid) {
     let content = i18n.t('registrations.list.payment_requested_on', { date: getRegistrationTimestamp(registeredOn) });
     let trigger = <span>{i18n.t('registrations.list.not_paid')}</span>;
 
@@ -151,6 +151,7 @@ export default function TableRow({
               <RegistrationTime
                 timestamp={timestamp}
                 paidOn={updatedAt}
+                hasPaid={hasPaid}
                 registeredOn={registeredOn}
                 paymentStatuses={paymentStatuses}
                 usesPaymentIntegration={competitionInfo['using_payment_integrations?']}