rockykitamura/grunt-po2json

grunt-po2json's Issues

NPM gives warning because Grunt is out of date

Installing this project from NPM currently gives out the following warning:

npm WARN [email protected] requires a peer of grunt@~0.4.1 but none was installed.

This is because the version in NPM still has Grunt version ~0.4.1 declared in the package.json file.

Could you update the package.json version to 0.3.1 and publish a new version to NPM? This would fix the issue. You should be able to publish the package just by running:

npm publish

Also pull request #19 can be closed, since the Grunt version is already updated.

NPM version doesn't contain stringOnly property

Commit 7f8188d isn't included into NPM version of grunt-po2json.

Allow msgctxt to be key only - no concatenation with string

Ideally it should be possible for this string in the .po file:

msgctxt "favouriteArtists.favouriteFtuMessage"
msgid "Added to your favourites."
msgstr "Dodano do Twoich ulubionych."

To be accessed as the "favouriteArtists.favouriteFtuMessage" key only - this is what occurs currently:

"favouriteArtists.favouriteFtuMessage\u0004Added to your favourites.": [
null,
"Dodano do Twoich ulubionych."
],

Ideally there should be an option for this to be the output:

"favouriteArtists.favouriteFtuMessage": [
null,
"Dodano do Twoich ulubionych."
],

Please publish this version

When doing a npm install I got an old version of this plugin with no po2json dependency and deprecated call to parseSync which both have been fixed.
This is kind of a problem in production.

Stopped instaling via npm today 11.09.2014 - package unpublished

Have you done any release recently? Today npm stopped installing grunt-po2json.
Tested on three machines, no other changes in dev config in the meanwhile

EDIT: npm registry says you unpublished the package. What might be a workaround?

grunt-po2json
This package has been unpublished.

Description Convert PO files to JSON using Grunt.
Unpublished By
rkitamura
Unpublished On Wed Sep 10 2014 23:21:42 GMT+0000 (UTC)

$ npm install grunt-po2json
npm WARN package.json [email protected] No repository field.
npm http GET http://registry.npmjs.org/grunt-po2json
npm http 404 http://registry.npmjs.org/grunt-po2json
npm ERR! TypeError: Cannot read property 'latest' of undefined
npm ERR! at next (/usr/share/npm/lib/cache.js:687:35)
npm ERR! at /usr/share/npm/lib/cache.js:675:5
npm ERR! at saved (/usr/share/npm/node_modules/npm-registry-client/lib/get.js:142:7)
npm ERR! at /usr/lib/nodejs/graceful-fs/polyfills.js:133:7
npm ERR! at Object.oncomplete (fs.js:107:15)
npm ERR! If you need help, you may report this log at:
npm ERR! http://github.com/isaacs/npm/issues
npm ERR! or email it to:
npm ERR! [email protected]

npm ERR! System Linux 3.13.0-34-generic
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install" "grunt-po2json"
npm ERR! cwd /home/alex/Documents/src
npm ERR! node -v v0.10.25
npm ERR! npm -v 1.3.10
npm ERR! type non_object_property_load
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR! /home/alex/Documents/src/npm-debug.log
npm ERR! not ok code 0

Doesn't convert pluralised strings correctly.

Polish po file:
msgctxt "product.trackCount"
msgid "%d song"
msgid_plural "%d songs"
msgstr[0] "%d utwór"
msgstr[1] "%d utwory"
msgstr[2] "%d utworów"

Polish json file:
"product.trackCount\u0004%d song": [
"%d songs",
"%d utwór",
"%d utwory",
"%d utworów"
],

I don't expect the "%d songs" to exist in the array after conversion - have I interpreted this wrongly or is this a problem.

Mention jed1.x format in documentation

Besides jed, po2json also accepts jed1.x format, which is not mentioned in documentation.

In fact, only after specifying jed1.x I was able to get my messages.js working with Jed v1.1.1.

The output of these two formats differs. In jed case, a leading null is inserted into a message specification array:

...
"Overload": [null, "Перегрузка"],
...

jed1.x:

...
"Overload": ["Перегрузка"],
...

The jed1.x format should definitely be mentioned in documentation.

Small issue in documentation

There's a small issue in documentation. Where it says:

  po2json: {
    options: {
        format: 'raw'
    }
    all: {
      src: ['test/**/*.po'],
      dest: 'some/destination/path/'
    }
  },
})

it should say:

grunt.initConfig({
  po2json: {
    options: {
        format: 'raw'
    },
    all: {
      src: ['test/**/*.po'],
      dest: 'some/destination/path/'
    }
  },
})

There's a missing ,

Publish current version to npm?

Will the current version be tagged and published on npm soon?

Using the file renaming feature is a nice thing and needed by my project. Currently I am fetching the version by its commit hash.

po2json should be a dependency

Hey,

I think po2json should be a dependency, not a devDependency.
I get the following error when running grunt po2json:

Running "po2json:all" (po2json) task
Warning: Cannot find module 'po2json' Use --force to continue.

Remove peerDeps

As of npm3+, peerDeps will no longer be installed automatically. Lets remove the peerDep grunt and keep the grunt devDep.

@rockykitamura