Thanks for the module! It would help to have an API method that returns the correct URI. Currently we use: $uri = _endicia_in_test_mode() ? ENDICIA_SANDBOX_LS_URI : ENDICIA_PRODUCTION_LS_URI; to get the active uri. The constants changed in the dev version and creating the method makes it easier to use in modules that extend the functionality.

CommentFileSizeAuthor
#1 endicia-module-2442093.patch949 byteschrisrockwell
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

chrisrockwell’s picture

chrisrockwell CreditAttribution: chrisrockwell commented
FileSize
endicia-module-2442093.patch949 bytes
arulan_pari’s picture

arulan_pari CreditAttribution: arulan_pari as a volunteer and commented

Hi chris,

I think you forget to mention the values for those variables (ENDICIA_SANDBOX_LS_URI and ENDICIA_PRODUCTION_LS_URI) am i right?

chrisrockwell’s picture

chrisrockwell CreditAttribution: chrisrockwell commented
Status: Active » Needs review

@Arulan Pari - no, those are defined at the top of endicia.module. It's been so long since I worked on this and I don't even have access to the codebase/site any longer so I can't quite remember what I was doing :).

However, it should be "Needs Review" so that maintainers can close it, accept it, or give feedback.