wikimedia/mediawiki-extensions-VisualEditor
1
0
Fork 0
mirror of https://gerrit.wikimedia.org/r/mediawiki/extensions/VisualEditor synced 2024-11-30 17:14:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rewrite a conditional to make the logic a little clearer.

Browse source 
I spent a couple minutes looking at this before I saw that we needed
`this.updateVisibility()` to only fire if the loop iterated at least
once.

Change-Id: I7d03f73a35e3ded539898effa064dc0e14ba595f
This commit is contained in:
Zoë 2024-07-08 15:27:07 +01:00
parent 12ef5b7471
commit 652e140dc5
1 changed files with 13 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
modules/ve-mw/ui/widgets/ve.ui.MWTocWidget.js
View file

@ -96,11 +96,11 @@ ve.ui.MWTocWidget.prototype.onMetaListRemove = function ( metaItem ) {
* Initialize TOC based on the presence of magic words * Initialize TOC based on the presence of magic words
*/ */
ve.ui.MWTocWidget.prototype.initFromMetaList = function () { ve.ui.MWTocWidget.prototype.initFromMetaList = function () {
const items = this.metaList.getItemsInGroup( 'mwTOC' ), const items = this.metaList.getItemsInGroup( 'mwTOC' );
len = items.length; if ( items.length === 0 ) {
let i = 0; return;
if ( len > 0 ) { }
for ( ; i < len; i++ ) { for ( let i = 0; i < items.length; i++ ) {
if ( items[ i ] instanceof ve.dm.MWTOCMetaItem ) { if ( items[ i ] instanceof ve.dm.MWTOCMetaItem ) {
const property = items[ i ].getAttribute( 'property' ); const property = items[ i ].getAttribute( 'property' );
if ( property === 'mw:PageProp/forcetoc' ) { if ( property === 'mw:PageProp/forcetoc' ) {
@ -112,7 +112,6 @@ ve.ui.MWTocWidget.prototype.initFromMetaList = function () {
} }
} }
this.updateVisibility(); this.updateVisibility();
}
}; };
/** /**