As part of getting custom languages working for Drupal 6, I had to fix a previous error I made by
associating languages with packages instead of with instances of packages.

This would mean that if you had the default profile on several platforms, but only have a certain language available on one of those platforms, it would show you the language even though your platform didn't have it.

Just need to change the populate function to accept both the platform and the profile as input, use that to get the iid in hosting_package_instance, and then get the languages using that iid from hosting_package_languages.

This involves some changes to the jquery.

CommentFileSizeAuthor
#1 hosting.oct_.25.x.patch8.45 KBxiong

Comments

xiong’s picture

xiong commented
Status: Active » Needs review
StatusFileSize
new hosting.oct_.25.x.patch8.45 KB

It's working now, please review.

adrian’s picture

adrian commented

I fixed 2 small problems in this patch, but it works perfectly now.

Namely it was showing language codes instead of the language names, and it wasn't sending the correct platform to the backend call (missing a :checked in the js, but added now)

committed.

adrian’s picture

adrian commented
Status: Needs review » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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