Skip to content

Multilingual Content (PoC)#6275

Closed
Cyperghost wants to merge 38 commits into6.2from
6.2-multilingual-content
Closed

Multilingual Content (PoC)#6275
Cyperghost wants to merge 38 commits into6.2from
6.2-multilingual-content

Conversation

@Cyperghost
Copy link
Copy Markdown
Contributor

@Cyperghost Cyperghost commented Apr 29, 2025 •
edited
Loading

Closes #6109

TODO

  • Migration script for the current rankTitle into wcf1_user_rank_content
    • Check if rankTitle is still used somewhere
    • Can we remove the old column rankTitle?
  • Ensure that UserRankCache is used to output the user rank if possible
  • BadgeColorFormField must be changed to work with more then one field
  • Check if the changes in UserProfile::getRank() do not cause any major problems in external dependencies/plugins
  • Migration script for the current buttonLabel into wcf1_bbcode_content
    • Check if buttonLabel is still used somewhere
    • Can we remove the old column buttonLabel?
  • Add eager cache for BB Codes
  • Rework for the shared_multipleLanguageInputJavascript template?

Cyperghost added 18 commits April 28, 2025 10:33
@Cyperghost
Add database table wcf1_user_rank_content
4251fea
@Cyperghost
Create new container MultilingualContainer to handle multilingual a…
8328c44 
…nd monolingual form fields
@Cyperghost
Create form data processor for processing multilingual and monolingua…
e1b8852 
…l values from `MultilingualContainer`
@Cyperghost
Add new column isMultilingual to wcf1_user_rank
649158f
@Cyperghost
Save multilingual values into wcf1_user_rank_content
9e9fdc3
@Cyperghost
Add todo for later
f691627
@Cyperghost
Rename to MultilingualFormContainer
11c36de
@Cyperghost
Load the title when editing
09e690a
@Cyperghost
Implement a user rank list that loads the title from `wcf1_user_rank_…
d4fc68a 
…content`
@Cyperghost
Implement a cache for the ViewableUserRank
7117f7f
@Cyperghost
Use ViewableUserRank and ViewableUserRankList for the interactions
e163973
@Cyperghost
Load title from wcf1_user_rank_content instead of using rankTitle
87a7adb
@Cyperghost
Change template type to IFormChildNode to allow also IFormContainer
023c76c
@Cyperghost
Allow appending multilingual form fields into a container
0aeac84
@Cyperghost
Check whether it is an IFormContainer instead of a FormContainer.
0f50d58
@Cyperghost
Implement helper functions to insert new form fields into the `Multil…
c522871 
…ingualFormContainer`
@Cyperghost
Extracting the code for loading multilingual content in `TMultilingua…
390da2f 
…lContentList`
@Cyperghost
Add languageID property to ViewableUserRank
bd0e44d
@Cyperghost Cyperghost requested a review from dtdesign April 29, 2025 12:00
dtdesign
dtdesign requested changes Apr 29, 2025
View reviewed changes
Comment thread wcfsetup/install/files/lib/data/TMultilingualContentList.class.php Outdated
Comment thread wcfsetup/install/files/lib/acp/form/UserRankAddForm.class.php Outdated
Comment thread wcfsetup/install/files/lib/acp/form/UserRankAddForm.class.php Outdated
Comment thread wcfsetup/install/files/acp/database/update_com.woltlab.wcf_6.2.php Outdated
Comment thread wcfsetup/install/files/acp/database/update_com.woltlab.wcf_6.2.php Outdated
Comment thread wcfsetup/install/files/lib/data/user/rank/UserRank.class.php Outdated
Comment thread wcfsetup/install/files/lib/data/user/rank/UserRankEditor.class.php Outdated
Comment thread wcfsetup/install/files/lib/data/user/rank/UserRankList.class.php Outdated
Comment thread wcfsetup/install/files/lib/data/user/rank/ViewableUserRank.class.php Outdated
Comment thread wcfsetup/install/files/lib/data/user/rank/ViewableUserRankList.class.php Outdated
Cyperghost and others added 8 commits April 30, 2025 08:34
@Cyperghost Cyperghost requested a review from dtdesign April 30, 2025 07:37
@Cyperghost Cyperghost closed this May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dtdesign dtdesign Awaiting requested review from dtdesign

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Cyperghost @dtdesign