? ? Pending

User tests: Successful: Unsuccessful:

avatar prakhar3062
prakhar3062
9 Feb 2022

Pull Request for Issue #36957

COM_CONTACT_EMAIL issue created by [dzandut]

76 ">

Testing Instructions

Enable email display in contact form and then enter the any value and you will see the value

Actual result BEFORE applying this Pull Request

Instead of value of variable , variable name will appear

Expected result AFTER applying this Pull Request

Value of email will appear when you enable email display

Documentation Changes Required

Not at all

avatar prakhar3062 prakhar3062 - open - 9 Feb 2022
avatar prakhar3062 prakhar3062 - change - 9 Feb 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Feb 2022
Category Front End com_contact
avatar rjharishabh rjharishabh - change - 9 Feb 2022
Labels Added: ? ?
avatar rjharishabh
rjharishabh - comment - 9 Feb 2022

@prakhar3062 please provide the test instructions and also add the issue number.

avatar prakhar3062
prakhar3062 - comment - 9 Feb 2022

Issue
#36957 COM_CONTACT_EMAIL in default_address.php.
Test Instructions
Enable email display in contact form and then enter the any value and you will see the value

avatar rjharishabh
rjharishabh - comment - 9 Feb 2022

@prakhar3062 Please update the PR description

avatar brianteeman
brianteeman - comment - 9 Feb 2022

This is only partially complete. The language string is still missing

avatar prakhar3062 prakhar3062 - change - 9 Feb 2022
The description was changed
avatar prakhar3062 prakhar3062 - edited - 9 Feb 2022
avatar prakhar3062
prakhar3062 - comment - 9 Feb 2022

@rjharishabh sorry for trouble i have updated it now.

avatar prakhar3062
prakhar3062 - comment - 9 Feb 2022

This is only partially complete. The language string is still missing

It is giving expected result in my system

avatar brianteeman
brianteeman - comment - 9 Feb 2022

Do you see COM_CONTACT_EMAIL when you view the source on the rendered page?

avatar prakhar3062 prakhar3062 - change - 9 Feb 2022
Labels
avatar joomla-cms-bot joomla-cms-bot - edited - 9 Feb 2022
avatar rjharishabh rjharishabh - change - 9 Feb 2022
The description was changed
Title
Resolve Com_contact_email bug in default_address.php
[4.1] Resolve Com_contact_email bug in default_address.php
avatar joomla-cms-bot joomla-cms-bot - edited - 9 Feb 2022
avatar prakhar3062
prakhar3062 - comment - 9 Feb 2022

Do you see COM_CONTACT_EMAIL when you view the source on the rendered page?

Yes sir

avatar brianteeman
brianteeman - comment - 9 Feb 2022

That means there is a language string missing then

avatar chmst
chmst - comment - 14 Feb 2022

@prakhar3062 would you like to complete this PR?

You can either change your language key to COM_CONTACT_EMAIL_LABEL, - this will be translated.

Or you can add a new line here for COM_CONTACT_EMAIL
https://github.com/joomla/joomla-cms/blob/4.1-dev/language/en-GB/com_contact.ini#L28

avatar prakhar3062 prakhar3062 - change - 15 Feb 2022
Labels Added: ?
avatar prakhar3062
prakhar3062 - comment - 15 Feb 2022

COM_CONTACT_EMAIL_LABEL

Thanks for reminding actually i was suffering with illness. Now i will try to contribute daily

avatar brianteeman brianteeman - test_item - 15 Feb 2022 - Tested successfully
avatar brianteeman
brianteeman - comment - 15 Feb 2022

I have tested this item successfully on ade3f25


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36976.

avatar Quy Quy - test_item - 15 Feb 2022 - Tested successfully
avatar Quy
Quy - comment - 15 Feb 2022

I have tested this item successfully on b881367


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36976.

avatar Quy Quy - alter_testresult - 15 Feb 2022 - brianteeman: Tested successfully
avatar Quy Quy - change - 15 Feb 2022
Status Pending Ready to Commit
avatar Quy
Quy - comment - 15 Feb 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36976.

avatar prakhar3062
prakhar3062 - comment - 16 Feb 2022

RTC

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36976.

RTC???

avatar rjharishabh
rjharishabh - comment - 16 Feb 2022

RTC means this PR is ready to merge in the Joomla Core.
RTC - Ready To Commit

avatar prakhar3062
prakhar3062 - comment - 16 Feb 2022

RTC means this PR is ready to merge in the Joomla Core. RTC - Ready To Commit

okay thanks for telling me

avatar Quy Quy - change - 28 Feb 2022
Labels Added: ?
Removed: ? ?
avatar Quy Quy - change - 28 Feb 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-02-28 16:23:48
Closed_By Quy
avatar Quy Quy - close - 28 Feb 2022
avatar Quy Quy - merge - 28 Feb 2022
avatar Quy
Quy - comment - 28 Feb 2022

Thanks.

avatar 7u5a
7u5a - comment - 14 Feb 2023

When adding COM_CONTACT_EMAIL it's now showing "> after the translation...


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36976.

avatar 7u5a 7u5a - test_item - 14 Feb 2023 - Tested unsuccessfully
avatar 7u5a
7u5a - comment - 14 Feb 2023

I have tested this item ? unsuccessfully on 68f2c93

When adding COM_CONTACT_EMAIL it's now showing "> after the translation...


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/36976.

avatar chmst
chmst - comment - 16 Feb 2023

This is an old PR and was merged in 4.1.
You should be on 4.2.7 now, so that's a new issue. could you open a new one please? And give some more information about the language you are using and your exact Joomla Version.

Add a Comment

Login with GitHub to post a comment