django-cfg 1.4.20__py3-none-any.whl → 1.4.23__py3-none-any.whl

django_cfg/apps/payments/views/api/balances.py

@@ -4,25 +4,24 @@ Balance ViewSets for the Universal Payment System v2.0.
 DRF ViewSets for balance and transaction management with service integration.
 """
 
-from rest_framework import viewsets, permissions, status
-from rest_framework.decorators import action
-from rest_framework.response import Response
-from django_filters.rest_framework import DjangoFilterBackend
 from django.contrib.auth import get_user_model
 from django.db import models
 from django.utils import timezone
+from rest_framework import permissions, status
+from rest_framework.decorators import action
+from rest_framework.response import Response
 
-from .base import PaymentBaseViewSet, NestedPaymentViewSet, ReadOnlyPaymentViewSet
-from ...models import UserBalance, Transaction
-from ...services import get_balance_service
+from django_cfg.modules.django_logging import get_logger
+
+from ...models import Transaction, UserBalance
 from ..serializers.balances import (
-    UserBalanceSerializer,
-    TransactionSerializer,
+    BalanceStatsSerializer,
     BalanceUpdateSerializer,
     FundsTransferSerializer,
-    BalanceStatsSerializer,
+    TransactionSerializer,
+    UserBalanceSerializer,
 )
-from django_cfg.modules.django_logging import get_logger
+from .base import NestedPaymentViewSet, ReadOnlyPaymentViewSet
 
 User = get_user_model()
 logger = get_logger("balance_viewsets")
@@ -34,24 +33,24 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
     
     Read-only access to user balances with statistics.
     """
-    
+
     queryset = UserBalance.objects.all()
     serializer_class = UserBalanceSerializer
     permission_classes = [permissions.IsAuthenticated]
     filterset_fields = ['user']
     search_fields = ['user__username', 'user__email']
     ordering_fields = ['balance_usd', 'created_at', 'updated_at']
-    
+
     def get_queryset(self):
         """Filter by user permissions and optimize queryset."""
         queryset = super().get_queryset().select_related('user')
-        
+
         # Non-staff users can only see their own balance
         if not self.request.user.is_staff:
             queryset = queryset.filter(user=self.request.user)
-        
+
         return queryset
-    
+
     @action(detail=True, methods=['post'])
     def add_funds(self, request, pk=None):
         """
@@ -60,14 +59,14 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
         POST /api/balances/{id}/add_funds/
         """
         balance = self.get_object()
-        
+
         # Permission check: users can only add funds to their own balance
         if not request.user.is_staff and balance.user != request.user:
             return Response(
                 {'error': 'You can only add funds to your own balance'},
                 status=status.HTTP_403_FORBIDDEN
             )
-        
+
         serializer = BalanceUpdateSerializer(
             data=request.data,
             context={
@@ -75,13 +74,13 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
                 'user_pk': balance.user.id
             }
         )
-        
+
         if serializer.is_valid():
             result = serializer.save()
             return Response(result)
-        
+
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
-    
+
     @action(detail=True, methods=['post'])
     def withdraw_funds(self, request, pk=None):
         """
@@ -90,19 +89,19 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
         POST /api/balances/{id}/withdraw_funds/
         """
         balance = self.get_object()
-        
+
         # Permission check
         if not request.user.is_staff and balance.user != request.user:
             return Response(
                 {'error': 'You can only withdraw from your own balance'},
                 status=status.HTTP_403_FORBIDDEN
             )
-        
+
         # Convert to negative amount for withdrawal
         data = request.data.copy()
         if 'amount' in data and data['amount'] > 0:
             data['amount'] = -abs(data['amount'])
-        
+
         serializer = BalanceUpdateSerializer(
             data=data,
             context={
@@ -110,13 +109,13 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
                 'user_pk': balance.user.id
             }
         )
-        
+
         if serializer.is_valid():
             result = serializer.save()
             return Response(result)
-        
+
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
-    
+
     @action(detail=True, methods=['post'])
     def transfer_funds(self, request, pk=None):
         """
@@ -125,25 +124,25 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
         POST /api/balances/{id}/transfer_funds/
         """
         balance = self.get_object()
-        
+
         # Permission check
         if not request.user.is_staff and balance.user != request.user:
             return Response(
                 {'error': 'You can only transfer from your own balance'},
                 status=status.HTTP_403_FORBIDDEN
             )
-        
+
         serializer = FundsTransferSerializer(
             data=request.data,
             context=self.get_serializer_context()
         )
-        
+
         if serializer.is_valid():
             result = serializer.save()
             return Response(result)
-        
+
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
-    
+
     @action(detail=False, methods=['get'])
     def analytics(self, request):
         """
@@ -152,13 +151,13 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
         GET /api/balances/analytics/?days=30
         """
         serializer = BalanceStatsSerializer(data=request.query_params)
-        
+
         if serializer.is_valid():
             result = serializer.save()
             return Response(result)
-        
+
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
-    
+
     @action(detail=False, methods=['get'])
     def summary(self, request):
         """
@@ -168,7 +167,7 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
         """
         try:
             queryset = self.filter_queryset(self.get_queryset())
-            
+
             summary = queryset.aggregate(
                 total_users=models.Count('id'),
                 total_balance=models.Sum('balance_usd'),
@@ -182,7 +181,7 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
                     filter=models.Q(balance_usd=0)
                 ),
             )
-            
+
             return Response({
                 'summary': {
                     **summary,
@@ -191,7 +190,7 @@ class UserBalanceViewSet(ReadOnlyPaymentViewSet):
                 },
                 'generated_at': timezone.now().isoformat()
             })
-            
+
         except Exception as e:
             logger.error(f"Balance summary failed: {e}")
             return Response(
@@ -206,24 +205,24 @@ class TransactionViewSet(ReadOnlyPaymentViewSet):
     
     Read-only access to transaction history with filtering.
     """
-    
+
     queryset = Transaction.objects.all()
     serializer_class = TransactionSerializer
     permission_classes = [permissions.IsAuthenticated]
     filterset_fields = ['user', 'transaction_type', 'payment_id']
     search_fields = ['description', 'payment_id']
     ordering_fields = ['created_at', 'amount']
-    
+
     def get_queryset(self):
         """Filter by user permissions and optimize queryset."""
         queryset = super().get_queryset().select_related('user')
-        
+
         # Non-staff users can only see their own transactions
         if not self.request.user.is_staff:
             queryset = queryset.filter(user=self.request.user)
-        
+
         return queryset
-    
+
     @action(detail=False, methods=['get'])
     def by_type(self, request):
         """
@@ -233,14 +232,14 @@ class TransactionViewSet(ReadOnlyPaymentViewSet):
         """
         try:
             queryset = self.filter_queryset(self.get_queryset())
-            
+
             type_stats = {}
             for type_choice in Transaction.TransactionType.choices:
                 type_code = type_choice[0]
                 type_name = type_choice[1]
-                
+
                 type_transactions = queryset.filter(transaction_type=type_code)
-                
+
                 type_stats[type_code] = {
                     'name': type_name,
                     'total_transactions': type_transactions.count(),
@@ -255,19 +254,19 @@ class TransactionViewSet(ReadOnlyPaymentViewSet):
                         )['avg'] or 0
                     ),
                 }
-            
+
             return Response({
                 'type_stats': type_stats,
                 'generated_at': timezone.now().isoformat()
             })
-            
+
         except Exception as e:
             logger.error(f"Transaction type stats failed: {e}")
             return Response(
                 {'error': f'Type stats failed: {e}'},
                 status=status.HTTP_500_INTERNAL_SERVER_ERROR
             )
-    
+
     @action(detail=False, methods=['get'])
     def recent(self, request):
         """
@@ -278,19 +277,19 @@ class TransactionViewSet(ReadOnlyPaymentViewSet):
         try:
             limit = int(request.query_params.get('limit', 10))
             limit = min(limit, 100)  # Cap at 100
-            
+
             queryset = self.filter_queryset(self.get_queryset())
             recent_transactions = queryset.order_by('-created_at')[:limit]
-            
+
             serializer = self.get_serializer(recent_transactions, many=True)
-            
+
             return Response({
                 'transactions': serializer.data,
                 'count': len(serializer.data),
                 'limit': limit,
                 'generated_at': timezone.now().isoformat()
             })
-            
+
         except Exception as e:
             logger.error(f"Recent transactions failed: {e}")
             return Response(
@@ -305,33 +304,33 @@ class UserTransactionViewSet(NestedPaymentViewSet):
     
     User-scoped access to transaction history.
     """
-    
+
     queryset = Transaction.objects.all()
     serializer_class = TransactionSerializer
     permission_classes = [permissions.IsAuthenticated]
     filterset_fields = ['transaction_type', 'payment_id']
     search_fields = ['description', 'payment_id']
     ordering_fields = ['created_at', 'amount']
-    
+
     # Nested ViewSet configuration
     parent_lookup_field = 'user_pk'
     parent_model_field = 'user'
-    
+
     # Read-only operations only
     http_method_names = ['get', 'head', 'options']
-    
+
     def get_queryset(self):
         """Filter by user and optimize queryset."""
         queryset = super().get_queryset()
-        
+
         # Additional permission check: users can only see their own transactions
         if not self.request.user.is_staff:
             user_id = self.kwargs.get('user_pk')
             if str(self.request.user.id) != str(user_id):
                 return queryset.none()
-        
+
         return queryset
-    
+
     @action(detail=False, methods=['get'])
     def summary(self, request, user_pk=None):
         """
@@ -341,7 +340,7 @@ class UserTransactionViewSet(NestedPaymentViewSet):
         """
         try:
             queryset = self.filter_queryset(self.get_queryset())
-            
+
             summary = queryset.aggregate(
                 total_transactions=models.Count('id'),
                 total_credits=models.Sum(
@@ -354,14 +353,14 @@ class UserTransactionViewSet(NestedPaymentViewSet):
                 ),
                 net_amount=models.Sum('amount_usd'),
             )
-            
+
             # Get type breakdown
             type_breakdown = dict(
                 queryset.values('transaction_type')
                 .annotate(count=models.Count('id'))
                 .values_list('transaction_type', 'count')
             )
-            
+
             return Response({
                 'user_id': user_pk,
                 'summary': {
@@ -373,7 +372,7 @@ class UserTransactionViewSet(NestedPaymentViewSet):
                 },
                 'generated_at': timezone.now().isoformat()
             })
-            
+
         except Exception as e:
             logger.error(f"User transaction summary failed: {e}")
             return Response(

django_cfg/apps/payments/views/api/base.py

@@ -4,17 +4,17 @@ Base ViewSet classes for the Universal Payment System v2.0.
 Common functionality for all payment system ViewSets.
 """
 
-from rest_framework import viewsets, permissions, status
-from rest_framework.decorators import action
-from rest_framework.response import Response
-from django_filters.rest_framework import DjangoFilterBackend
-from rest_framework.filters import SearchFilter, OrderingFilter
-from rest_framework.exceptions import NotFound
+from datetime import timedelta
+from typing import Any, Dict
 
-from django.db.models import Count, Sum, Avg, Q
+from django.db.models import Avg, Count, Sum
 from django.utils import timezone
-from datetime import timedelta
-from typing import Dict, Any
+from django_filters.rest_framework import DjangoFilterBackend
+from rest_framework import permissions, status, viewsets
+from rest_framework.decorators import action
+from rest_framework.exceptions import NotFound
+from rest_framework.filters import OrderingFilter, SearchFilter
+from rest_framework.response import Response
 
 from django_cfg.modules.django_logging import get_logger
 
@@ -28,15 +28,15 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
     Provides standard CRUD operations plus common actions like stats,
     health checks, and optimized querysets.
     """
-    
+
     permission_classes = [permissions.IsAuthenticated]
     filter_backends = [DjangoFilterBackend, SearchFilter, OrderingFilter]
     ordering = ['-created_at']
     versioning_class = None  # Disable versioning for payments API
-    
+
     # Serializer classes mapping for different actions
     serializer_classes = {}
-    
+
     def get_queryset(self):
         """
         Optimized queryset with select_related and prefetch_related.
@@ -44,13 +44,13 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
         Override in subclasses to add specific optimizations.
         """
         queryset = super().get_queryset()
-        
+
         # Add common optimizations
         if hasattr(self.queryset.model, 'user'):
             queryset = queryset.select_related('user')
-        
+
         return queryset
-    
+
     def get_serializer_class(self):
         """
         Dynamic serializer selection based on action.
@@ -59,7 +59,7 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
         """
         serializer_classes = getattr(self, 'serializer_classes', {})
         return serializer_classes.get(self.action, self.serializer_class)
-    
+
     def get_serializer_context(self):
         """
         Enhanced serializer context with additional data.
@@ -69,18 +69,18 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
             'action': self.action,
             'user': self.request.user,
         })
-        
+
         # Add object ID for detail actions
         if self.action in ['retrieve', 'update', 'partial_update', 'destroy']:
             context['object_id'] = self.kwargs.get('pk')
-        
+
         # Add parent object ID for nested routes
         for key, value in self.kwargs.items():
             if key.endswith('_pk'):
                 context[key] = value
-        
+
         return context
-    
+
     @action(detail=False, methods=['get'])
     def stats(self, request, **kwargs):
         """
@@ -90,35 +90,35 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
         """
         try:
             queryset = self.filter_queryset(self.get_queryset())
-            
+
             # Basic counts
             total_count = queryset.count()
-            
+
             stats = {
                 'total_count': total_count,
                 'generated_at': timezone.now().isoformat(),
             }
-            
+
             # Add status breakdown if model has status field
             if hasattr(queryset.model, 'status'):
                 stats['status_breakdown'] = self._get_status_breakdown(queryset)
-            
+
             # Add amount summary if model has amount fields
             if hasattr(queryset.model, 'amount_usd'):
                 stats['amount_summary'] = self._get_amount_summary(queryset)
-            
+
             # Add time-based breakdown
             stats['time_breakdown'] = self._get_time_breakdown(queryset)
-            
+
             return Response(stats)
-            
+
         except Exception as e:
             logger.error(f"Stats generation failed: {e}")
             return Response(
                 {'error': f'Stats generation failed: {e}'},
                 status=status.HTTP_500_INTERNAL_SERVER_ERROR
             )
-    
+
     @action(detail=False, methods=['get'])
     def health(self, request, **kwargs):
         """
@@ -128,7 +128,7 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
         """
         try:
             queryset = self.get_queryset()
-            
+
             # Basic health metrics
             health_data = {
                 'service': self.__class__.__name__,
@@ -137,16 +137,16 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
                 'model': queryset.model.__name__,
                 'timestamp': timezone.now().isoformat(),
             }
-            
+
             # Check recent activity (last 24 hours)
             if hasattr(queryset.model, 'created_at'):
                 recent_count = queryset.filter(
                     created_at__gte=timezone.now() - timedelta(hours=24)
                 ).count()
                 health_data['recent_activity'] = recent_count
-            
+
             return Response(health_data)
-            
+
         except Exception as e:
             logger.error(f"Health check failed: {e}")
             return Response(
@@ -158,7 +158,7 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
                 },
                 status=status.HTTP_503_SERVICE_UNAVAILABLE
             )
-    
+
     def _get_status_breakdown(self, queryset) -> Dict[str, int]:
         """Get status breakdown for statistics."""
         return dict(
@@ -166,7 +166,7 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
             .annotate(count=Count('id'))
             .values_list('status', 'count')
         )
-    
+
     def _get_amount_summary(self, queryset) -> Dict[str, Any]:
         """Get amount summary for statistics."""
         aggregates = queryset.aggregate(
@@ -174,20 +174,20 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
             average_amount=Avg('amount_usd'),
             count=Count('id')
         )
-        
+
         return {
             'total_amount_usd': float(aggregates['total_amount'] or 0),
             'average_amount_usd': float(aggregates['average_amount'] or 0),
             'transaction_count': aggregates['count'],
         }
-    
+
     def _get_time_breakdown(self, queryset) -> Dict[str, int]:
         """Get time-based breakdown for statistics."""
         if not hasattr(queryset.model, 'created_at'):
             return {}
-        
+
         now = timezone.now()
-        
+
         return {
             'last_24h': queryset.filter(
                 created_at__gte=now - timedelta(hours=24)
@@ -199,7 +199,7 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
                 created_at__gte=now - timedelta(days=30)
             ).count(),
         }
-    
+
     def handle_exception(self, exc):
         """
         Enhanced exception handling with logging.
@@ -209,7 +209,7 @@ class PaymentBaseViewSet(viewsets.ModelViewSet):
             'user_id': getattr(self.request.user, 'id', None) if hasattr(self, 'request') else None,
             'exception_type': type(exc).__name__,
         })
-        
+
         return super().handle_exception(exc)
 
 
@@ -219,30 +219,30 @@ class ReadOnlyPaymentViewSet(PaymentBaseViewSet):
     
     Provides list, retrieve, and stats actions only.
     """
-    
+
     http_method_names = ['get', 'head', 'options']
-    
+
     def create(self, request, *args, **kwargs):
         """Disable create action."""
         return Response(
             {'error': 'Create operation not allowed'},
             status=status.HTTP_405_METHOD_NOT_ALLOWED
         )
-    
+
     def update(self, request, *args, **kwargs):
         """Disable update action."""
         return Response(
             {'error': 'Update operation not allowed'},
             status=status.HTTP_405_METHOD_NOT_ALLOWED
         )
-    
+
     def partial_update(self, request, *args, **kwargs):
         """Disable partial update action."""
         return Response(
             {'error': 'Update operation not allowed'},
             status=status.HTTP_405_METHOD_NOT_ALLOWED
         )
-    
+
     def destroy(self, request, *args, **kwargs):
         """Disable destroy action."""
         return Response(
@@ -257,10 +257,10 @@ class NestedPaymentViewSet(PaymentBaseViewSet):
     
     Automatically filters queryset by parent object and sets parent on creation.
     """
-    
+
     parent_lookup_field = 'user_pk'  # Override in subclasses
     parent_model_field = 'user'      # Override in subclasses
-    
+
     def get_queryset(self):
         """Filter queryset by parent object from URL."""
         queryset = super().get_queryset()
@@ -275,7 +275,7 @@ class NestedPaymentViewSet(PaymentBaseViewSet):
             queryset = queryset.filter(**filter_kwargs)
 
         return queryset
-    
+
     def perform_create(self, serializer):
         """Set parent object when creating nested resource."""
         parent_id = self.kwargs.get(self.parent_lookup_field)
@@ -283,7 +283,7 @@ class NestedPaymentViewSet(PaymentBaseViewSet):
             # Get parent model class
             parent_field = getattr(self.queryset.model, self.parent_model_field)
             parent_model = parent_field.field.related_model
-            
+
             try:
                 parent_obj = parent_model.objects.get(id=parent_id)
                 serializer.save(**{self.parent_model_field: parent_obj})
@@ -291,13 +291,13 @@ class NestedPaymentViewSet(PaymentBaseViewSet):
                 raise NotFound(f"Parent object not found: {parent_id}")
         else:
             serializer.save()
-    
+
     def get_serializer_context(self):
         """Add parent object ID to serializer context."""
         context = super().get_serializer_context()
-        
+
         parent_id = self.kwargs.get(self.parent_lookup_field)
         if parent_id:
             context[self.parent_lookup_field] = parent_id
-        
+
         return context