sahana / vesuvius

Sahana Vesuvius
http://sahanafoundation.org/products/vesuvius/
MIT License
24 stars 27 forks source link

Contexts Still Visible #4

Closed ravihansa3000 closed 10 years ago

ravihansa3000 commented 10 years ago

Internal reference : https://jira.nlm.nih.gov:8443/browse/PL-352

See attached screen shot.

Also, please notice all the notices I receive in the console.


Imported from Launchpad using lp2gh.

ravihansa3000 commented 10 years ago

(by triune)

ravihansa3000 commented 10 years ago

(by rasade88) Fixed. Please test. (I had messed up the check for whether the "|" is in each string. I was actually separating each msgid into a list if there WASN'T a "|" character in the string, whereas it should've been the other way around.) This patch fixes bug #816129 as well.

ravihansa3000 commented 10 years ago

(by triune) Problem still persists.

ravihansa3000 commented 10 years ago

(by rasade88) Are you getting this with the latest repository? Because I'm not.

ravihansa3000 commented 10 years ago

(by rasade88) Greg, Not to really annoy you, but I'm still not getting the bug even with the code off the repo. Might it be something to do with the PHP configuration on your server, as before? Can you send me the conf file on pl.nlm.nih.gov so that I can take a look at it? Thanks.

On Thu, Aug 4, 2011 at 10:41 PM, Ramindu Deshapriya <

ravihansa3000 commented 10 years ago

(by triune) You're not annoying me at all, but this bug sure is ;)

Conf file attached.

-G

On Wed, Sep 14, 2011 at 04:39, Ramindu Deshapriya <

wrote:

Greg, Not to really annoy you, but I'm still not getting the bug even with the code off the repo. Might it be something to do with the PHP configuration on your server, as before? Can you send me the conf file on pl.nlm.nih.gov so that I can take a look at it? Thanks.

On Thu, Aug 4, 2011 at 10:41 PM, Ramindu Deshapriya <

▴triune.

ravihansa3000 commented 10 years ago

(by rasade88) Greg, I've been going through our code and working towards fixing this bug. No committable code yet, but just thought I'd let you know I'm still on the job. :) Thanks.

On Thu, Sep 15, 2011 at 1:11 AM, Greg Miernicki

ravihansa3000 commented 10 years ago

(by rasade88) This is what I get under Hospital Admin with the conf file you sent me:

I still don't get why this is happening.

ravihansa3000 commented 10 years ago

(by triune) That is Hospital Admin 2.0.... which was pushed live last Friday. I completely re-wrote the module from scratch, so the contexts were tossed out. You would have to manually re-add one somewhere to see the difference.

Glad you're still on the job :)

-G

On Tue, Sep 20, 2011 at 15:45, Ramindu Deshapriya <

wrote:

This is what I get under Hospital Admin with the conf file you sent me:

I still don't get why this is happening.

\ Attachment added: "hospital_admin.png"

https://bugs.launchpad.net/vesuvius/+bug/817698/+attachment/2433020/+files/hospital_admin.png

You received this bug notification because you are subscribed to the bug report. https://bugs.launchpad.net/bugs/817698

Title: Contexts Still Visible

Status in Sahana Vesuvius: Confirmed

Bug description: See attached screen shot.

Also, please notice all the notices I receive in the console.

To manage notifications about this bug go to: https://bugs.launchpad.net/vesuvius/+bug/817698/+subscriptions

▴triune.

ravihansa3000 commented 10 years ago

(by rasade88) Oh, I see. I'll add the contexts and see how that pans out. :)

BTW, the new version is nice, you're very nimble with AJAX, and I've learned quite a bit from your 'xajax' scheme as well.

ravihansa3000 commented 10 years ago

(by triune) Thank you, the only problem is that the Vesuvius theme is becoming very far out of sync with the lpf3 theme with all the recent changes. We will have to re-derive the vesuvius theme before the next packaged version of the product.

On Wed, Sep 21, 2011 at 01:48, Ramindu Deshapriya <

wrote:

Oh, I see. I'll add the contexts and see how that pans out. :)

BTW, the new version is nice, you're very nimble with AJAX, and I've learned quite a bit from your 'xajax' scheme as well.

You received this bug notification because you are subscribed to the bug report. https://bugs.launchpad.net/bugs/817698

Title: Contexts Still Visible

Status in Sahana Vesuvius: Confirmed

Bug description: See attached screen shot.

Also, please notice all the notices I receive in the console.

To manage notifications about this bug go to: https://bugs.launchpad.net/vesuvius/+bug/817698/+subscriptions

▴triune.

ravihansa3000 commented 10 years ago

(by rasade88) Greg, Are there any changes that should be made to .htaccess? Because I can't seem to access the login. Thanks.

On Wed, Sep 21, 2011 at 7:46 PM, Greg Miernicki

ravihansa3000 commented 10 years ago

(by triune) Attached the latest .htaccess and example files.

On Sun, Oct 2, 2011 at 11:05, Ramindu Deshapriya

ravihansa3000 commented 10 years ago

(by rasade88) Thanks. did you manage to re-test? I'm not getting anything when I go to Hospital Administration.

ravihansa3000 commented 10 years ago

(by triune) Re-testing shows this bug to still be a problem.

ravihansa3000 commented 10 years ago

(by rasade88) Lovely, I'll take another look.

ravihansa3000 commented 10 years ago

(by rasade88) Greg,

Can I get an update on exactly what this bug is doing now? I mean, is it the contexts being displayed under all locales, only under en_US or under a different specific locale?

Thanks.

ravihansa3000 commented 10 years ago

(by rasade88) Fixed in Translation Branch. Please test.

ravihansa3000 commented 10 years ago

(by rasade88) Fix applied: Context added to "People Locator" header in header.php. The translated string appears without the context when a locale is chosen. (Google Translate should be disabled).

ravihansa3000 commented 10 years ago

(by triune) Ramindu, I just merged in yur branch to the trunk and I still see the context regardless of whether or not $conf['enable_locale'] is true or false. Screenshot attached.

ravihansa3000 commented 10 years ago

(by triune)

ravihansa3000 commented 10 years ago

(by triune) I pushed my merged branch here for you to test: https://code.launchpad.net/~triune/vesuvius/transMerged

ravihansa3000 commented 10 years ago

(by rasade88) Thanks Greg, I'll take a look tomorrow and get back to you.

ravihansa3000 commented 10 years ago

(by rasade88) Greg, I think the problem here is that the correct .po file hasn't been pushed from my branch. Maybe .po files are in your bzrignore or maybe on my branch (not the case in my local repo, though). The *.po's on the merged branch are outdated code-wise. Anyway, I'm attaching my fr.po file to enable immediate testing until I can figure out the Bazaar anomaly.

Thanks.

ravihansa3000 commented 10 years ago

(by rasade88) Upon realizing that all my *.po files are needed, I'm attaching my complete /res/locale folder.

ravihansa3000 commented 10 years ago

(by rasade88) Along with a screenshot of my installation of Vesuvius, which shows the translations being applied: