Skip to content

Fix SSL orders in WHMCS 8.x - #84

Open
orudge wants to merge 1 commit into
namecheap:masterfrom
orudge:fix-whmcs-8
Open

Fix SSL orders in WHMCS 8.x#84
orudge wants to merge 1 commit into
namecheap:masterfrom
orudge:fix-whmcs-8

Conversation

@orudge

@orudge orudge commented Dec 7, 2020

Copy link
Copy Markdown

Fixes SSL orders in WHMCS 8.x.

This was referenced Dec 7, 2020
@jesussuarz

Copy link
Copy Markdown

Hi

thanks for posting,

Could you explain exactly what it fixes?

some video or screenshot to see what problem it solves.

@orudge

orudge commented Dec 7, 2020

Copy link
Copy Markdown
Author

I don’t have a screenshot handy, but after an SSL order is placed, if you press the Create button in the WHMCS admin area, it appears to succeed. It fails though, because it doesn’t create the item in the tblsslorders table, as new fields have been added and don’t appear to have default values associated with them. You don’t know that it’s failed until the end user clicks on the configuressl.php e-mail link, which leads to a page stating that “the link you clicked is invalid” (or similar).

@jesussuarz

Copy link
Copy Markdown

I don’t have a screenshot handy, but after an SSL order is placed, if you press the Create button in the WHMCS admin area, it appears to succeed. It fails though, because it doesn’t create the item in the tblsslorders table, as new fields have been added and don’t appear to have default values associated with them. You don’t know that it’s failed until the end user clicks on the configuressl.php e-mail link, which leads to a page stating that “the link you clicked is invalid” (or similar).


very well,

I have applied the fix to my files.

Have you ever had client-side issues downloading the .zip file?

I can download the zip from the client area, however the zip is corrupted.

Could you tell me if you have this same problem?

@orudge

orudge commented Dec 7, 2020

Copy link
Copy Markdown
Author

This seems to work fine for me. I’m using nginx if that makes any difference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants