Add `.current` method back to ShopifyAPI::Shop and ShopifyAPI::RecurringApplicationCharge

[scart88]

Issue summary

With the new breaking api change, ShopifyAPI::Shop.current, ShopifyAPI::RecurringApplicationCharge.current methods are missing. Is there any plan in adding these back?

Expected behavior

ShopifyAPI::Shop.current should return the current shop from the session.

Actual behavior

Right now it returns
NoMethodError: undefined method 'current' for ShopifyAPI::Shop:Class

Previous implementation

def current
  (all || []).find { |c| c.status == 'active' }
end

Steps to reproduce the problem

Shop.first.with_shopify_session do
  puts ShopifyAPI::Shop.current
end 

Loading development environment (Rails 7.0.2.3)
[1] pry(main)> Shop.first.with_shopify_session { ShopifyAPI::Shop.current }
  Shop Load (7.4ms)  SELECT "shops".* FROM "shops" ORDER BY "shops"."id" ASC LIMIT $1  [["LIMIT", 1]]
NoMethodError: undefined method `current' for ShopifyAPI::Shop:Class
from (pry):1:in `block in __pry__'
[2] pry(main)> 

Specifications

• shopify_api version: 10.0.0
• Shopify API version used (example: 2022-04): 2022-04

[mllemango]

We're adding this to the TODO list! Keeping this issue open to let others also chime in their feedback

[uurcank]

@mllemango this is a breaking change was not mentioned in docs. It has been widely used so please add it back.

[BentoumiTech]

Do there is any similar method that it has been replaced with ?

[stuartchaney]

+1, this was the last hurdle that prevented us from upgrading version :(

[marisveide]

+1 from me as well.

[kaarelss]

+1 from me as well

[jaredbeck]

I, also, do not see any mention of this breaking change in the changelog. Will the changelog be updated?

Until this method is restored, what is the workaround?

[axlekb]

This works: ShopifyAPI::Shop.all[0]

[asecondwill]

@axlekb @mllemango - does this workaround really work reliably? It isnt very descriptive. At first glance it could just get the first shop that has this app or something.

Like the others above, I used the ShopifyAPI::Shop.current method a lot. Upgrading to shopify_app 20+ and so shopify API 10+ is being presented as mandatory by the approvals team, including delisting our app and saying we have to update to be re-evaluated. So it's pretty important this stuff is either put right or workarounds / changes properly documented.

[sativva]

I'm using that instead, waiting for upgrase. xo

ShopifyAPI::Shop.all(session: @session).first

[fluke]

ShopifyAPI::RecurringApplicationCharge.current can be replaced with (ShopifyAPI::RecurringApplicationCharge.all || []).find { |c| c.status == 'active' }

[immakdas]

Sometimes I think the dev team in Shopify has no idea what they are doing. Instead of improving, things became harder to handle. Current app version is v21 but you still have to write tons of custom code to handle payments, trial/ decline payments etc

[sagar-ranglani]

This works: ShopifyAPI::Shop.all[0]

Does this really work?

[kirillplatonov]

The old method was very convenient. Not sure what was the motivation for removing it during the rewrite. For now, I added this to my Rails initializer to restore old behavior:

# config/initializers/shopify_app.rb

Rails.application.config.after_initialize do
  class ShopifyAPI::Shop
    def self.current(session: ShopifyAPI::Context.active_session, fields: nil)
      ShopifyAPI::Shop.all(session: session, fields: fields).first
    end
  end
end