Comments

bucefal91’s picture

Category: Task » Feature request
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new2.93 KB

hello!

I know it's been quite some time since you asked about it. But in case you still need this, here's a patch that does decent job of integrating Currency module into Units.

This patch introduces a new submodule: units_currency. After you enable it, a new "Currency" measure will be automatically created and it will be filled in with all the currencies that are available from Currency module. All the conversion logic within this "Currency" measure will be executed exactly the same way as it happens in Currency module, i.e. if in Currency module 100 EUR converts into 120 USD, then you'll receive the same 120 USD if you convert 100 EUR within Units module.

bucefal91’s picture

StatusFileSize
new3.01 KB

I just had a look at my code again and found that Units would import even disabled currencies, which is not the desired behavior. I have just fixed it. I am going to commit it soon.

  • bucefal91 committed 10e5de6 on 7.x-1.x
    Issue #1956544 by bucefal91: Bridging Currency module into Units.
    
  • bucefal91 committed ec315d7 on 7.x-1.x
    Issue #1956544 by bucefal91: only import enabled currencies from the...
bucefal91’s picture

Status: Needs review » Fixed

The issue is now through.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.