#90 Calendar Add/Edit Event buttons translation

4.1.2
closed-accepted
nobody
None
5
2014-09-01
2014-08-11
Esteban Monge
No

When a recurring event is modified the buttons of popup yello box or "Warning" do not translate in any language

I modified the file $INSTALLATION_PATH/openemr/interface/main/calendar/add_edit_event.php and replaced these lines:

<input type="button" name="all_events" id="all_events" value="  All  ">
<input type="button" name="future_events" id="future_events" value="Future">
<input type="button" name="current_event" id="current_event" value="Current">
<input type="button" name="recurr_cancel" id="recurr_cancel" value="Cancel">

To:

<input type="button" name="all_events" id="all_events" value="<?php echo xlt('All')?>">
<input type="button" name="future_events" id="future_events" value="<?php echo xlt('Future')?>">
<input type="button" name="current_event" id="current_event" value="<?php echo xlt('Current')?>">
<input type="button" name="recurr_cancel" id="recurr_cancel" value="<?php echo xlt('Cancel')?>">

I don't know if spaces in "All" word are for formatting matters, but in Spanish looks great.

Additionally I needed to add a new Constant in Administration->Other->Language->Add Constant for "Future" word with Spanish translation to "Futuro"

Thanks a lot for your great work.

Discussion

  • Brady Miller
    Brady Miller
    2014-08-17

    Hi Estaban,

    Thanks for the bug report. It's better to bring in fixes via git/github, and there is a tutorial here to set up a github repo in openemr:
    http://www.open-emr.org/wiki/index.php/Git_for_dummies

    This also allows to review/test code easily. For example, should be using the xla() function and not the xlt() function in that place.

    No need to do more work on this, however; I'll bring it in and fix the xlt() to xla(). Just send me an email at brady@sparmy.com with your author name and email, so I can give you credit on the commit (or if you don't want name/email attached to commit, then let me know and I'll bring it in under my name/email).

    Thanks,
    -brady
    OpenEMR

     
  • Brady Miller
    Brady Miller
    2014-09-01

    Hi,

    I just committed your fix to the official codebase. Thanks for the contribution!!

    -brady
    OpenEMR

     
  • Brady Miller
    Brady Miller
    2014-09-01

    • status: open --> closed-accepted