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

Bug in LicenseQuery #11

Closed
putyourlightson opened this issue Jul 25, 2018 · 6 comments
Closed

Bug in LicenseQuery #11

putyourlightson opened this issue Jul 25, 2018 · 6 comments
Assignees

Comments

@putyourlightson
Copy link

This line is throwing a "column not found" error, it should be:

$this->subQuery->andWhere(Db::parseParam('digitalproducts_licenses.productId', $this->productId));

Version: 2.0.2

@lazlo-bonin
Copy link

Same for orderId, it should be:

        if ($this->orderId) {
            $this->subQuery->andWhere(Db::parseParam('digitalproducts_licenses.orderId', $this->orderId));
        }

@lazlo-bonin
Copy link

Is this plugin abandoned @andris-sevcenko ? This issue and plenty of others are pretty big but easy to fix. Considering forking the repository just to fix them.

@andris-sevcenko
Copy link
Contributor

@lazlo-bonin it's not abandoned - I've been pretty busy last few months with Craft 3.1 and the project config stuff. The most likely scenario is I resume working on this after the Dot All conference to get this plugin ready for 3.1, which will include addressing all the issues here as well.

@andris-sevcenko
Copy link
Contributor

Also, pull requests are always welcome. We have finite resources as well and appreciate the help :)

@lazlo-bonin
Copy link

I apologize for coming off a bit harsh, I understand these things take time. I just hadn't seen a post from a contributor on this repo in a while and was wondering if it was wise to continue to use it in production.

My own resources are spread pretty thin as well unfortunately, but if I can find the time I'll try forking and submitting a few PRs.

@andris-sevcenko andris-sevcenko self-assigned this Sep 21, 2018
@andris-sevcenko
Copy link
Contributor

@lazlo-bonin not offended in the slightest. I should have reacted at least, even if I was busy.

I'm guessing I'll be resuming work on this in two weeks or so. If your timeline is more pressing, you can just hot-patch your install and submit PR at your leisure. There's no pressure.

This can definitely be used in production. The only possible way this plugin ends its life I can imagine at this point if it just gets rolled into Commerce itself.

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

No branches or pull requests

2 participants