Comments (5)
PR #193 opened. This should cover all locales that don't use "#" based postal codes. Tests and Integration tests pass. Added a new test for en-CA postal codes.
from kotlin-faker.
Hi @efenderbosch-atg ,
Thanks for opening this issue!
Yes, you're likely right that en-GB locale needs to be handled separately, like the nl
one. I really haven't gone through all of the localizations to find these kinds of issues, and only fix them when I or someone else encounters it :)
Would you like to submit a PR with a fix?
from kotlin-faker.
I can probably do that. I'm wondering if the when
is required at all? Would this simply cover all use cases?
fun postcode() = with(fakerService) { resolve("postcode").generexify().replace("/", "") }
from kotlin-faker.
I can probably do that. I'm wondering if the
when
is required at all? Would this simply cover all use cases?
fun postcode() = with(fakerService) { resolve("postcode").generexify().replace("/", "") }
Up to you , but if you want to contribute - you're welcome to do so :)
No, it won't. Most postcodes aren't in the form of regex, but use #
chars as a placeholder for a digit (e.g.
else
handles.Also I assume that generating strings from regexes is probably (haven't really tested it) slower than just returning a string and replacing
#
with a random digit.from kotlin-faker.
This should be available in latest snapshot after the build finishes.
Otherwise I'll make a new RC version later this week.
Thanks for your contribution
from kotlin-faker.
Related Issues (20)
- Add context when newInstance() fails HOT 4
- Crashes in Jetpack Compose previews HOT 1
- Move to java17 as default
- Cannot generate valid en-GB postcode HOT 7
- Remove SLF4J HOT 1
- Getting 'ClassCastException' when using kotlin-faker with Generics Type HOT 9
- Add blns
- Add rendering for sub-provider functions in cli
- Make it possible to lookup providers by name in cli
- Improve integration tests
- Allow RandomClassProvider::randomClassInstance to use predefinedGenerators from config if present HOT 4
- RandomClassProvider doesn't handle constructorless types correctly in all instances HOT 1
- Allow random to be used with unique HOT 1
- Regexify generates invalid value HOT 2
- Regexify should take Regex as an argument HOT 1
- Regexify fails with StackOverflowError HOT 5
- random.nextLong(bound: Long) also returns negative values HOT 3
- Suggestion: internet.ipv4() and internet.ipv6() HOT 4
- Support inner classes in RandomClassProvider
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from kotlin-faker.