-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding the Multilanguage Associations Manager #13537
Merged
wilsonge
merged 41 commits into
joomla:staging
from
rdeutz:com_associations_rewrite_merge
Jan 17, 2017
Merged
Changes from 9 commits
Commits
Show all changes
41 commits
Select commit
Hold shift + click to select a range
0fbc25a
Merge Associations rewrite
rdeutz f341aad
updated searchtool with the new way
rdeutz 79d95a5
udpated edit view title
rdeutz 7cf3996
added contact associationshelper class
rdeutz c1ebda1
temp fix
rdeutz b662491
fix for category filter
rdeutz f7c85a5
added newsfeeds associations helper
rdeutz 0495886
CAPS for params
rdeutz ebe4afc
lang tag and added a helper function
rdeutz 00eb70b
added land tags
rdeutz a54d3f7
code style fix
rdeutz f7ebff9
better title in associations view
rdeutz 87a3138
better title
rdeutz 100fb42
use the usual naming
rdeutz 7b0b75f
fix language tag, thanks to brian teeman and twitter :-)
rdeutz 417f0fc
Merge branch 'com_associations_rewrite_merge' of https://github.com/r…
zero-24 f50ad78
initial review
zero-24 b2fda8a
on simple change
zero-24 67d6000
on simple change 2
zero-24 90f7639
simple
zero-24 f5ce260
some more helper changes
zero-24 31a0185
Update associations.php
zero-24 55dc10c
Update associations.php
zero-24 0826d30
Merge pull request #7 from zero-24/com_associations_rewrite_merge
rdeutz 7e8422b
app isn’t set a model property
rdeutz 1ca1c8e
correct return value
rdeutz dbba275
simplify code adn use helper method
rdeutz fc79b46
use typename directly
rdeutz d64c742
changed the tooltip position
rdeutz edccd27
Correct menu helper
infograf768 868edd4
remove unreacable code
rdeutz ca50fd1
correcting checked_out
infograf768 04df8c3
com_menus
infograf768 7ab1191
Merge pull request #8 from infograf768/patch-10
rdeutz e5fb8bd
Merge pull request #9 from infograf768/patch-11
rdeutz 97c437f
Merge pull request #11 from infograf768/patch-13
rdeutz 406d18d
fixed not supportted message
rdeutz e4f03ee
installation
rdeutz cc61448
fix menu install
rdeutz 95fc4b6
Merge branch 'staging' into com_associations_rewrite_merge
rdeutz 9fd1d9b
Spaces -> tabs
wilsonge File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,7 +2,7 @@ | |
<extension type="component" version="3.7" method="upgrade"> | ||
<name>com_associations</name> | ||
<author>Joomla! Project</author> | ||
<creationDate>September 2016</creationDate> | ||
<creationDate>Januar 2017</creationDate> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
<copyright>(C) 2005 - 2016 Open Source Matters. All rights reserved.</copyright> | ||
<license>GNU General Public License version 2 or later; see LICENSE.txt</license> | ||
<authorEmail>admin@joomla.org</authorEmail> | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does
$itemType->realcomponent
exists? or is a leftover?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, the lang string
COM_ASSOCIATIONS_COMPONENT_NOT_SUPPORTED
is not present in the ini file.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we should have something like:
throw new Exception(JText::sprintf('COM_ASSOCIATIONS_COMPONENT_NOT_SUPPORTED', JText::_($extensionName), 404);
and
COM_ASSOCIATIONS_COMPONENT_NOT_SUPPORTED="The extension %s does not support multilingual associations."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed with 406d18d thanks!