Remove unavailable LoremPixel generator

[saiqulhaq]

LoremPixel is no longer an available service. An option is to use LoremFlickr whenever LoremPixel is referenced in the codebase.

Before we remove it, we need to add a deprecation for users.

Steps

• Add a deprecation warning that LoremPixel is going to be removed in the next release. See Gem::Deprecate docs for reference. - Merged in Deprecate LoremPixel #2590
• Use LoremFlickr whenever LoremPixel was being used, such as here. This will fix the deprecation warnings we see on our specs - Merged in used LoremFlickr wherever LoremPixel is used #2614
• Remove deprecated LoremPixel generator. This can only be done after December 2022.

[stefannibrasil]

HI @saiqulhaq could you please provide more information about this issue? I am not sure I understand what is the problem and what is your proposal. Thanks!

[saiqulhaq]

we need to delete the class
because loremflickr.com is down

[stefannibrasil]

I can access LoremFlickr normally. Could you try again on your side? What exactly is the error you're getting when running Faker? Thanks!

[saiqulhaq]

sorry I meant lorem pixel

[psibi]

Yeah, makes sense to delete it since it's not up anymore:

faker/lib/faker/default/lorem_pixel.rb

Line 61 in 15a8bc3

url_parts << ['lorempixel.com']

[stefannibrasil]

oh, okay, it makes more sense now. It looks like Lorem Pixel is now Lorem Ipsum.

At first glance, LoremPixel seems to be used only here. Could we use LoremFlickr instead to get rid of LoremPixel (and subsequent TODOs in that file)?

[yagosansz]

Hey folks, would it be okay if I started working on this one?

[thdaraujo]

Hey folks, would it be okay if I started working on this one?

yes, thank you! 👏

[stefannibrasil]

I believe we need to add a deprecation warning before following along with this removal. I am going to update the issue with the steps needed for this.

[uzorjchibuzor]

I'll pick this up.

[stefannibrasil]

thank you @uzorjchibuzor the first two tasks could be done in two initial PRs.

[uzorjchibuzor]

@stefannibrasil I'll make sure to mention you in the PRs. So this issue can remain open until all the tasks are done.

[uzorjchibuzor]

Add Tests for LoremFlickr on test_lorem_flickr.rb

@stefannibrasil I am not sure I understand this part, shouldn't this say that I should add a test on lorem_pixel instead that when is it called to refer to the lorem_flickr?

Also, all of the previous assertions on test_lorem_flickr are not relevant anymore as calls to that class are forwarded to LoremFlickr. What I am asking is that, after I test for the referral, is there anything more to add?

[stefannibrasil]

Hi @uzorjchibuzor I left a code review on your PR, thank you! About the tests, you're right. LoremFlickr already has tests. I removed this step from the issue.

[stefannibrasil]

The merged PR fixes the first step of this issue. Reopening to complete the remaining steps.

[uzorjchibuzor]

Thank you.

[stefannibrasil]

@uzorjchibuzor I updated the issue to reflect the next steps. The next one, to replace LoremPixel with LoremFlickr on faker, can be started anytime. Let me know if you have any questions. Thank you!

[uzorjchibuzor]

@stefannibrasil No questions.

[italopires]

Hi there!

I saw the last free step, I just opened a PR to try complete this issue. Please fill free to review the last one.

[stefannibrasil]

hi @italopires thank you for your interest in contributing to Faker! This issue is already assigned to @uzorjchibuzor. We were just waiting on the deprecation message date to arrive to complete the work.

@uzorjchibuzor We can move on with this issue because December is here 🎄 Do you want to open a PR or contribute with @italopires ? I just want to make sure we are collaborating. Thank you, both!

[italopires]

Hi @stefannibrasil and @uzorjchibuzor, my apologies. It was my first attempt to contribute open source. I will close the PR and try another issue, in case you need it, I can reopen. Thank you very much!!

[stefannibrasil]

@uzorjchibuzor are you still interested in finishing the issue?

[uzorjchibuzor]

I will take care of this today. Sorry for the delay. I had to clear some backlog.

…

On Tue, 20 Dec 2022 at 5:26 AM, Stefanni Brasil ***@***.***> wrote: @uzorjchibuzor <https://github.com/uzorjchibuzor> are you still interested in finishing the issue? — Reply to this email directly, view it on GitHub <#2505 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHZNEUXLT3WRBCOFSXJW523WOEYNZANCNFSM52QR5BLA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[stefannibrasil]

@uzorjchibuzor no worries! I didn't mean to rush you, I just wanted to know if you still had the bandwidth. Thank you!

[vbrazo]

I've just googled LoremPixel and it has indeed disappeared from the web. That was a good catch!

I tried to access the LoremFlickr website but my Avast is complaining..

Another alternative to LoremFlickr and LoremPixel would be Picsum:

• https://picsum.photos/
• https://picsum.photos/200/200