Skip to content
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

Remove Message:exists #39

Merged
merged 1 commit into from
Jan 15, 2024
Merged

Remove Message:exists #39

merged 1 commit into from
Jan 15, 2024

Conversation

HamishSlater
Copy link
Contributor

Expensive resulting in many queries

ERM34902

Expensive resulting in many queries

ERM34902
@HamishSlater HamishSlater requested a review from osnard January 11, 2024 08:40
@HamishSlater HamishSlater self-assigned this Jan 11, 2024
Copy link
Member

@osnard osnard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feature is required fir the MediaWiki Sidebar. We should rather use a modified version of SimpleTreeLinkNode in the book navigation panel

@HamishSlater
Copy link
Contributor Author

HamishSlater commented Jan 12, 2024

MediaWiki Sidebar doesn't seem to use SimpleTreeLinkNode.php or SimpleTreeTextNode.php; it uses SimpleCard.php, SimpleCardHeader.php.& SimpleLinklistGroupFromArray.php.

Update: Enhanced Sidebar does -> working on solution.

@HamishSlater
Copy link
Contributor Author

Parse Message key in Enhanced Sidebar
https://gerrit.wikimedia.org/r/c/mediawiki/skins/BlueSpiceDiscovery/+/990041

@DvogelHallowelt
Copy link
Contributor

Using a modified version of SimpleTreeLinkNode for the book tree-panel is overhead. It is much more easier to modify code like suggested and modify the sidebar implementation in skin

@osnard
Copy link
Member

osnard commented Jan 15, 2024

@osnard osnard merged commit 6566f3a into master Jan 15, 2024
1 check passed
@osnard osnard deleted the ERM34902 branch January 15, 2024 07:37
wmfgerrit pushed a commit to wikimedia/mediawiki-skins-BlueSpiceDiscovery that referenced this pull request Jan 15, 2024
Allows removal of Message:exists from commonuserinterface component
Expensive resulting in many queries
hallowelt/mwstake-mediawiki-component-commonuserinterface#39

ERM34902

[REL1_39, REL1_39-4.4.x, master]

Change-Id: I1dfebceebdb55b8083fa8463fbbe41278aa3618b
wmfgerrit pushed a commit to wikimedia/mediawiki-skins-BlueSpiceDiscovery that referenced this pull request Jan 15, 2024
Allows removal of Message:exists from commonuserinterface component
Expensive resulting in many queries
hallowelt/mwstake-mediawiki-component-commonuserinterface#39

ERM34902

[REL1_39, REL1_39-4.4.x, master]

Change-Id: I1dfebceebdb55b8083fa8463fbbe41278aa3618b
wmfgerrit pushed a commit to wikimedia/mediawiki-skins-BlueSpiceDiscovery that referenced this pull request Jan 15, 2024
Allows removal of Message:exists from commonuserinterface component
Expensive resulting in many queries
hallowelt/mwstake-mediawiki-component-commonuserinterface#39

ERM34902

[REL1_39, REL1_39-4.4.x, master]

Change-Id: I1dfebceebdb55b8083fa8463fbbe41278aa3618b
wmfgerrit pushed a commit to wikimedia/mediawiki-skins that referenced this pull request Jan 15, 2024
* Update BlueSpiceDiscovery from branch 'master'
  to 1ada1f164cbfd32e9b5680b0cf3a2305bad7d67b
  - Parse Message key
    
    Allows removal of Message:exists from commonuserinterface component
    Expensive resulting in many queries
    hallowelt/mwstake-mediawiki-component-commonuserinterface#39
    
    ERM34902
    
    [REL1_39, REL1_39-4.4.x, master]
    
    Change-Id: I1dfebceebdb55b8083fa8463fbbe41278aa3618b
osnard pushed a commit that referenced this pull request Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants