Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Server crashes when payment pending for ad #8

Open
frazzydee opened this issue Apr 8, 2021 · 1 comment
Open

Server crashes when payment pending for ad #8

frazzydee opened this issue Apr 8, 2021 · 1 comment
Labels
bug Something isn't working

Comments

@frazzydee
Copy link

First of all thank you for this amazing tool.

The other day, when experimenting with this newly installed package, I kept getting an error 5000.

I believe it was because I accidentally posted too many ads. That's when the problem started, and it resolved after I deleted it.

If that is the case, then when payment is pending on an ad (for example, if you exceed 2 ads in the Real Estate or Cars & Vehicles sections), the server throws an error.

Note the log: https://pastebin.com/Q7NevnV1

There would not be a page number if payment is pending (the ad isn't posted yet, but rather in a pending state).

Steps to reproduce: (have not tried a second time as I'm worried my account will get flagged given how many times I've done this already when experimenting)

  1. Post an ad in the real estate section.
  2. Do it again (now 2 ads)
  3. Do it again (now 3 ads)

Expected results: You are able to log into kijiji-manager and view your ad. Ideally it would note that the ad is "Payment pending" similar to the Kijiji website.

Actual results: Server throws an error 5000. You cannot do anything, including log out of your profile. Theoretically can be fixed if you delete the "payment pending" ad from the kijiji website.

@jackm
Copy link
Owner

jackm commented Apr 8, 2021

This is indeed an untested part of the code when it comes to Kijiji limiting the number of ads in a certain category or when payments to Kijiji come into play. This needs further investigation.

The issue when a page number does not exist for an ad is fixed in commit 0f62e0a

@jackm jackm added the bug Something isn't working label Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants