Attached is a screenshot, I will post a patch for comment #1. Essentially, I submit that LANGUAGE_NONE provides negative feedback to the majority of users even though it might be technically correct. I'll describe my fix once I submit the patch.

Default Language Issue

CommentFileSizeAuthor
#4 language-none-1346360-4.patch595 bytesjoshmiller
#1 languageDefault_1346360_01.patch1.5 KBjoshmiller
default_language_issue.PNG3.26 KBjoshmiller

Comments

joshmiller’s picture

joshmiller
he/him
Primary language English
Location Indianapolis, Indiana, USA
commented
Status: Active » Needs review
StatusFileSize
new languageDefault_1346360_01.patch1.5 KB

Attached is the proposed fix. It's possible we could also bring in all of the languages and pick the "first" enabled language, but I thought that might be overkill.

Thoughts?

rszrama’s picture

rszrama commented

Ahh, curious. Do you know if this is how Drupal handles nodes created through the interface as well?

In fact, I see that a product display created through the UI will show up as "Language neutral." Does your patch have the same effect?

rszrama’s picture

rszrama commented
Version: 7.x-1.0-rc5 » 7.x-1.x-dev
Status: Needs review » Needs work

Hmm, I tried the patch, but it just resulted in my nodes not having proper field data (and still showing as undefined language).

joshmiller’s picture

joshmiller
he/him
Primary language English
Location Indianapolis, Indiana, USA
commented
Status: Needs work » Needs review
StatusFileSize
new language-none-1346360-4.patch595 bytes

Good point. @raystuart here at the Commerce Guys conference helped me drush make --working-copy so that I could test my code snippets. Turns out that there isn't a sensible default for nodes #1336824: set node language to LANGUAGE_NONE in node_save()

Attached is the real fix this time. Fully tested, solved the problem. The generated nodes have the correct language neutral database entry that every other Drupal node has by default.

rszrama’s picture

rszrama commented
Status: Needs review » Fixed

Sweet, works as advertised. I still apparently don't know how to create a multilingual Drupal site, but at least turning on Locale + Content Translation showed me that our default nodes are Language neutral. I suppose I actually need a separate module to translate content?

Status: Fixed » Closed (fixed)

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