i18n fields Plugin
  1. i18n fields Plugin
  2. GPI18NFIELDS-9

The i18n-fields plugin does not work well in GGTS

    Details

    • Type: Bug Bug
    • Status: In Progress
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 0.6.1
    • Fix Version/s: None
    • Labels:
      None
    • Environment:
      OSX 10.8.3 Grails 2.2x

      Description

      The i18nfields.ConfigProvider class hard codes its config location to be System.properties['user.dir']. When run on the command line, user.dir would be the grails project, but when run in Eclipse, user.dir is where you launched Eclipse from. The issue is noted on the SpringSource Jira: https://issuetracker.springsource.com/browse/STS-3362

        Activity

        Hide
        Andrew Eisenberg added a comment -

        The following should work inside of ConfigProvider:

        	static final CONFIG_LOCATION = "${BuildSettingsHolder.settings.baseDir}/grails-app/conf/Config.groovy"
        

        I would also recommend adding a bit of error recovery in the getConfig method. If Config.groovy doesn't exist, it would be nice to fail gracefully with a proper error message, rather than to abort compilation. Inside of GGTS, the error message is almost incomprehensible.

        Show
        Andrew Eisenberg added a comment - The following should work inside of ConfigProvider : static final CONFIG_LOCATION = "${BuildSettingsHolder.settings.baseDir}/grails-app/conf/Config.groovy" I would also recommend adding a bit of error recovery in the getConfig method. If Config.groovy doesn't exist, it would be nice to fail gracefully with a proper error message, rather than to abort compilation. Inside of GGTS, the error message is almost incomprehensible.
        Hide
        Jorge Uriarte added a comment -

        Thanks for the exploration Will solve it and publish a new version as soon as possible.

        Show
        Jorge Uriarte added a comment - Thanks for the exploration Will solve it and publish a new version as soon as possible.
        Hide
        Jorge Uriarte added a comment -

        I did a 0.6.4 release, maybe you might want to test with that (sorry I don't really have bandwidth right now to test it throughfully, even though we're planning to tackle new features soon, driven by project needs)

        Show
        Jorge Uriarte added a comment - I did a 0.6.4 release, maybe you might want to test with that (sorry I don't really have bandwidth right now to test it throughfully, even though we're planning to tackle new features soon, driven by project needs)
        Hide
        Shawn Thomas added a comment -

        Thanks Jorge. I'll try it out and let you know if there are any problems.

        Show
        Shawn Thomas added a comment - Thanks Jorge. I'll try it out and let you know if there are any problems.
        Hide
        MichaelM added a comment -

        I have the same problem.

        Show
        MichaelM added a comment - I have the same problem.

          People

          • Assignee:
            Jorge Uriarte
            Reporter:
            Shawn Thomas
          • Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated: