Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions eventyay_paypal/apps.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from . import __version__

try:
from pretix.base.plugins import PluginConfig
from eventyay.base.plugins import PluginConfig
except ImportError:
raise RuntimeError("Python package 'paypal' is not installed.")
Comment on lines 8 to 9
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion (code-quality): Explicitly raise from a previous error (raise-from-previous-error)

Suggested change
except ImportError:
raise RuntimeError("Python package 'paypal' is not installed.")
except ImportError as e:
raise RuntimeError("Python package 'paypal' is not installed.") from e


Expand All @@ -14,7 +14,7 @@ class PaypalPluginApp(AppConfig):
name = "eventyay_paypal"
verbose_name = _("PayPal")

class PretixPluginMeta:
class EventyayPluginMeta:
name = _("PayPal")
author = "eventyay"
version = __version__
Expand Down
6 changes: 3 additions & 3 deletions eventyay_paypal/migrations/0002_initial.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,18 +8,18 @@ class Migration(migrations.Migration):

dependencies = [
('eventyay_paypal', '0001_initial'),
('pretixbase', '0001_initial'),
('base', '0001_initial'),
]

operations = [
migrations.AddField(
model_name='referencedpaypalobject',
name='order',
field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='pretixbase.order'),
field=models.ForeignKey(on_delete=django.db.models.deletion.CASCADE, to='base.order'),
),
migrations.AddField(
model_name='referencedpaypalobject',
name='payment',
field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, to='pretixbase.orderpayment'),
field=models.ForeignKey(null=True, on_delete=django.db.models.deletion.CASCADE, to='base.orderpayment'),
),
]
4 changes: 2 additions & 2 deletions eventyay_paypal/models.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,5 @@

class ReferencedPayPalObject(models.Model):
reference = models.CharField(max_length=190, db_index=True, unique=True)
order = models.ForeignKey('pretixbase.Order', on_delete=models.CASCADE)
payment = models.ForeignKey('pretixbase.OrderPayment', null=True, blank=True, on_delete=models.CASCADE)
order = models.ForeignKey('base.Order', on_delete=models.CASCADE)
payment = models.ForeignKey('base.OrderPayment', null=True, blank=True, on_delete=models.CASCADE)
24 changes: 12 additions & 12 deletions eventyay_paypal/payment.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,19 +17,19 @@
from django.utils.translation import gettext as __
from django.utils.translation import gettext_lazy as _
from i18nfield.strings import LazyI18nString
from pretix.base.decimal import round_decimal
from pretix.base.models import Event, Order, OrderPayment, OrderRefund, Quota
from pretix.base.payment import BasePaymentProvider, PaymentException
from pretix.base.services.mail import SendMailException
from pretix.base.settings import SettingsSandbox
from pretix.helpers.urls import build_absolute_uri as build_global_uri
from pretix.multidomain.urlreverse import build_absolute_uri
from eventyay.base.decimal import round_decimal
from eventyay.base.models import Event, Order, OrderPayment, OrderRefund, Quota
from eventyay.base.payment import BasePaymentProvider, PaymentException
from eventyay.base.services.mail import SendMailException
from eventyay.base.settings import SettingsSandbox
from eventyay.helpers.urls import build_absolute_uri as build_global_uri
from eventyay.multidomain.urlreverse import build_absolute_uri

from .models import ReferencedPayPalObject
from .paypal_rest import PaypalRequestHandler
from .utils import safe_get

logger = logging.getLogger("pretix.plugins.eventyay_paypal")
logger = logging.getLogger("eventyay.plugins.eventyay_paypal")

SUPPORTED_CURRENCIES = [
"AUD",
Expand Down Expand Up @@ -712,7 +712,7 @@ def execute_refund(self, refund: OrderRefund):
if errors := refund_payment.get("errors"):
logger.error("execute_refund: %s", errors["reason"])
refund.order.log_action(
"pretix.event.order.refund.failed",
"eventyay.event.order.refund.failed",
{
"local_id": refund.local_id,
"provider": refund.provider,
Expand All @@ -737,7 +737,7 @@ def execute_refund(self, refund: OrderRefund):

if errors := refund_detail.get("errors"):
refund.order.log_action(
"pretix.event.order.refund.failed",
"eventyay.event.order.refund.failed",
{
"local_id": refund.local_id,
"provider": refund.provider,
Expand All @@ -761,7 +761,7 @@ def execute_refund(self, refund: OrderRefund):
refund.save(update_fields=["state"])
else:
refund.order.log_action(
"pretix.event.order.refund.failed",
"eventyay.event.order.refund.failed",
{
"local_id": refund.local_id,
"provider": refund.provider,
Expand Down Expand Up @@ -873,7 +873,7 @@ def shred_payment_info(self, obj: Union[OrderPayment, OrderRefund]):

for le in (
obj.order.all_logentries()
.filter(action_type="pretix.plugins.eventyay_paypal.event")
.filter(action_type="eventyay.plugins.eventyay_paypal.event")
.exclude(data="")
):
d = le.parsed_data
Expand Down
2 changes: 1 addition & 1 deletion eventyay_paypal/paypal_rest.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
from cryptography.fernet import Fernet
from django.core.cache import cache

logger = logging.getLogger("pretix.plugins.eventyay_paypal")
logger = logging.getLogger("eventyay.plugins.eventyay_paypal")


class PaypalRequestHandler:
Expand Down
14 changes: 7 additions & 7 deletions eventyay_paypal/signals.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@
from django.dispatch import receiver
from django.template.loader import get_template
from django.utils.translation import gettext_lazy as _
from pretix.base.forms import SecretKeySettingsField
from pretix.base.signals import (logentry_display, register_global_settings,
from eventyay.base.forms import SecretKeySettingsField
from eventyay.base.signals import (logentry_display, register_global_settings,
register_payment_providers,
requiredaction_display)

Expand All @@ -19,7 +19,7 @@ def register_payment_provider(sender, **kwargs):

@receiver(signal=logentry_display, dispatch_uid="paypal_logentry_display")
def pretixcontrol_logentry_display(sender, logentry, **kwargs):
if logentry.action_type != 'pretix.plugins.eventyay_paypal.event':
if logentry.action_type != 'eventyay.plugins.eventyay_paypal.event':
return

data = json.loads(logentry.data)
Expand All @@ -44,16 +44,16 @@ def pretixcontrol_logentry_display(sender, logentry, **kwargs):

@receiver(signal=requiredaction_display, dispatch_uid="paypal_requiredaction_display")
def pretixcontrol_action_display(sender, action, request, **kwargs):
if not action.action_type.startswith('pretix.plugins.eventyay_paypal'):
if not action.action_type.startswith('eventyay.plugins.eventyay_paypal'):
return

data = json.loads(action.data)

if action.action_type == 'pretix.plugins.eventyay_paypal.refund':
if action.action_type == 'eventyay.plugins.eventyay_paypal.refund':
template = get_template('plugins/paypal/action_refund.html')
elif action.action_type == 'pretix.plugins.eventyay_paypal.overpaid':
elif action.action_type == 'eventyay.plugins.eventyay_paypal.overpaid':
template = get_template('plugins/paypal/action_overpaid.html')
elif action.action_type == 'pretix.plugins.eventyay_paypal.double':
elif action.action_type == 'eventyay.plugins.eventyay_paypal.double':
template = get_template('plugins/paypal/action_double.html')

ctx = {'data': data, 'event': sender, 'action': action}
Expand Down
2 changes: 1 addition & 1 deletion eventyay_paypal/urls.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
from django.urls import include
from django.urls import re_path as url
from pretix.multidomain import event_url
from eventyay.multidomain import event_url

from .views import (abort, oauth_disconnect, oauth_return, redirect_view,
success, webhook)
Expand Down
12 changes: 6 additions & 6 deletions eventyay_paypal/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,16 @@
from django.views.decorators.csrf import csrf_exempt
from django.views.decorators.http import require_POST
from django_scopes import scopes_disabled
from pretix.base.models import Event, Order, OrderPayment, OrderRefund, Quota
from pretix.base.payment import PaymentException
from pretix.control.permissions import event_permission_required
from pretix.multidomain.urlreverse import eventreverse
from eventyay.base.models import Event, Order, OrderPayment, OrderRefund, Quota
from eventyay.base.payment import PaymentException
from eventyay.control.permissions import event_permission_required
from eventyay.multidomain.urlreverse import eventreverse

from .models import ReferencedPayPalObject
from .payment import Paypal
from .utils import safe_get

logger = logging.getLogger("pretix.plugins.eventyay_paypal")
logger = logging.getLogger("eventyay.plugins.eventyay_paypal")


@xframe_options_exempt
Expand Down Expand Up @@ -360,7 +360,7 @@ def webhook(request, *args, **kwargs):
if order_detail is None or payment is None:
return HttpResponse("Order or payment not found", status=HTTPStatus.BAD_REQUEST)

payment.order.log_action("pretix.plugins.eventyay_paypal.event", data=event_json)
payment.order.log_action("eventyay.plugins.eventyay_paypal.event", data=event_json)

def handle_refund():
refund_id_in_event = safe_get(event_json, ["resource", "id"])
Expand Down