-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
QRCode namespace doesn't exist in .NET 8 #489
Comments
Note, the above code does not work in either .NET Core 6 OR 7. |
Hi @ScottDurkin , |
Yeah, for me too. I believe the problem was created with the 1.4.3 changes. |
Thank you for your response! I'll take a look! |
@cezar-pimentel Thanks for your offer. I would love to see such PR. But currently I'm going through all the older PRs and see if I can merge them. I fear if you start your PR now, the master will change a couple of times, while you're working on the PR. Maybe it's better to wait until the majority of the older PRs is merged. |
No worries, we can wait. There's no need to rush. Thank you my friend! 👊💪 |
@cezar-pimentel @ScottDurkin FYI, your issues are related to the fact that portions of QRCoder have been removed that rely on System.Drawing.Common when targeting .NET 6+ on Linux, as System.Drawing.Common is not supported in that scenario (and indeed would just throw You have a couple options for 1.4.3:
In addition, there are a few PRs that already exist that attempt to address to this problem:
Another solution would be to take a dependency on System.Drawing.Common for all platforms (which would include cc: @codebude |
I like the Supported Platform OS , but I would recommend the method signature stays consistent between them. Right now the number of parameters for each renderer is not the same, and a feature in one (like an overlay Icon), is not in the other. |
Having the method signature identical for each renderer may not be feasible, as the parameter types will be different per renderer. For example, a renderer based on System.Drawing.Common will use the |
Has the problem been solved? |
I'm trying to use Base64QRCode in .NET 8 and having the same issue, how could that be resolved? |
@codeputer never say never, but I'm pretty sure it can't be implemented that way. The whole philosophy behind the modular renderers is to get the best out of each target format/renderer. This is where the different method signatures come from. If the GetGraphic method of every renderer had the same signature, we would have to remove many great features. Example: Many renderers can be passed an image that is drawn as a logo on the QR code. However, the purely text-based ASCIIQrCode simply cannot draw images and will therefore never accept an image in the GetGraphic method. If we were to harmonize the method signatures, we would have to delete the logo functionality in all other renders. I don't think that makes sense.
@Seanxwy Yes and no. The comment by Shane32 explains very well why the QRCode renderer is not available in .NET8. (TL;DR; It is based on System.Drawing.Common, which is no longer supported by Microsoft). However, as Shane explained, you can use other renderers or target Windows only (
@nicetomytyuk this will be resolved with PR #495 which will be merged this week. The code changes will be part of QRCoder 1.5.0 that I plan to release soon. (Next couple of days.) |
Any plan to make QRCoder supported in .net8? |
Type of issue
[X ] Bug
[ ] Question (e.g. about handling/usage)
[ ] Request for new feature/improvement
Expected Behavior
Should be able to create new QRCode Object by passing in QRCodeData.
Current Behavior
QRCode namespace doesn't exist.
Steps to Reproduce (for bugs)
Upgrade to .NET Core 8
Your Environment
Visual Studio 2022 latest version with .NET Core 8.
Code sample that works in .NET Core 5
The text was updated successfully, but these errors were encountered: