SOFORT: Small improvement in code would do well

  • Posts: 2143
  • Thank you received: 747
8 years 4 months ago #244551

-- HikaShop version -- : 2.6.3

Hi HikaShop Team,

Due to a (solved) question in the German corner here , I'd like to propose a small improvement in the SOFORT payment plugin for inclusion with one of the next HikaShop versions.

Reason:
Thankfully, almost all text strings use language constants, allowing simple translation/replacement. Except one or two, which are seemingly "left-overs" in Dutch language.

Proposed change:
In /plugins/hikashoppayment/sofort/sofort.php change line 55 from

$order_text = "\r\n".JText::sprintf('betaling Feeen en ridders: order ',$order->order_id);
to e.g.
$order_text = "\r\n".JText::sprintf('SOFORT_PAYMENT_PURPOSE',$order->order_id);
and include a line in the English language file as default, such as
SOFORT_PAYMENT_PURPOSE="Payment for order %s"

I think it would also be nice to change line 159 in the same file from
$element->payment_description = 'Betalen met Credit card';
to e.g.
$element->payment_description = 'Pay with SOFORT';
or similar in English.

I haven't looked for more, but the above would be of good help for users outside the Dutch language regions.

Thanks for consideration!


Need help with customisations of layouts, style or other site development? PM me!
(Don't forget to turn on "E-mail notification of new messages" )
Last edit: 8 years 4 months ago by lousyfool. Reason: Link added

Please Log in or Create an account to join the conversation.

  • Posts: 26158
  • Thank you received: 4028
  • MODERATOR
8 years 4 months ago #244556

Hi,

See : github.com/HikaShop/hikashoppayment-sofort

The Sofort plugin has not been implemented by the team but it was included during a period and maintained by the team (for bug fixes).
Since some months, I started to extract plugins not made by us into github repository.

Feel free to propose "pull request" and modification for that plugin ; we will take a look and validate your pull request (so your patch will be directly integrated into the plugin).

Regards,


Jerome - Obsidev.com
HikaMarket & HikaSerial developer / HikaShop core dev team.

Also helping the HikaShop support team when having some time or couldn't sleep.
By the way, do not send me private message, use the "contact us" form instead.
The following user(s) said Thank You: lousyfool

Please Log in or Create an account to join the conversation.

  • Posts: 2143
  • Thank you received: 747
8 years 4 months ago #244593

Thanks Jerome,

I wasn't aware of it being on Github. But guess now I am :)

Might dig into it there soon.

Thanks again!


Need help with customisations of layouts, style or other site development? PM me!
(Don't forget to turn on "E-mail notification of new messages" )

Please Log in or Create an account to join the conversation.

Time to create page: 0.060 seconds
Powered by Kunena Forum