Index: trunk/phase3/skins/MonoBook.php |
— | — | @@ -115,7 +115,7 @@ |
116 | 116 | <h5><?php $this->msg('views') ?></h5> |
117 | 117 | <ul> |
118 | 118 | <?php foreach($this->data['content_actions'] as $key => $tab) { ?> |
119 | | - <li id="ca-<?php echo htmlspecialchars($key) ?>"<?php |
| 119 | + <li id="ca-<?php echo Sanitizer::escapeId($key) ?>"<?php |
120 | 120 | if($tab['class']) { ?> class="<?php echo htmlspecialchars($tab['class']) ?>"<?php } |
121 | 121 | ?>><a href="<?php echo htmlspecialchars($tab['href']) ?>"><?php |
122 | 122 | echo htmlspecialchars($tab['text']) ?></a></li> |
— | — | @@ -127,7 +127,7 @@ |
128 | 128 | <div class="pBody"> |
129 | 129 | <ul> |
130 | 130 | <?php foreach($this->data['personal_urls'] as $key => $item) { ?> |
131 | | - <li id="pt-<?php echo htmlspecialchars($key) ?>"<?php |
| 131 | + <li id="pt-<?php echo Sanitizer::escapeId($key) ?>"<?php |
132 | 132 | if ($item['active']) { ?> class="active"<?php } ?>><a href="<?php |
133 | 133 | echo htmlspecialchars($item['href']) ?>"<?php |
134 | 134 | if(!empty($item['class'])) { ?> class="<?php |
— | — | @@ -144,12 +144,12 @@ |
145 | 145 | </div> |
146 | 146 | <script type="<?php $this->text('jsmimetype') ?>"> if (window.isMSIE55) fixalpha(); </script> |
147 | 147 | <?php foreach ($this->data['sidebar'] as $bar => $cont) { ?> |
148 | | - <div class='portlet' id='p-<?php echo htmlspecialchars($bar) ?>'> |
| 148 | + <div class='portlet' id='p-<?php echo Sanitizer::escapeId($bar) ?>'> |
149 | 149 | <h5><?php $out = wfMsg( $bar ); if (wfEmptyMsg($bar, $out)) echo $bar; else echo $out; ?></h5> |
150 | 150 | <div class='pBody'> |
151 | 151 | <ul> |
152 | 152 | <?php foreach($cont as $key => $val) { ?> |
153 | | - <li id="<?php echo htmlspecialchars($val['id']) ?>"<?php |
| 153 | + <li id="<?php echo Sanitizer::escapeId($val['id']) ?>"<?php |
154 | 154 | if ( $val['active'] ) { ?> class="active" <?php } |
155 | 155 | ?>><a href="<?php echo htmlspecialchars($val['href']) ?>"><?php echo htmlspecialchars($val['text']) ?></a></li> |
156 | 156 | <?php } ?> |
— | — | @@ -194,7 +194,7 @@ |
195 | 195 | <?php } |
196 | 196 | if($this->data['feeds']) { ?> |
197 | 197 | <li id="feedlinks"><?php foreach($this->data['feeds'] as $key => $feed) { |
198 | | - ?><span id="feed-<?php echo htmlspecialchars($key) ?>"><a href="<?php |
| 198 | + ?><span id="feed-<?php echo Sanitizer::escapeId($key) ?>"><a href="<?php |
199 | 199 | echo htmlspecialchars($feed['href']) ?>"><?php echo htmlspecialchars($feed['text'])?></a> </span> |
200 | 200 | <?php } ?></li><?php |
201 | 201 | } |
Index: trunk/phase3/RELEASE-NOTES |
— | — | @@ -57,8 +57,8 @@ |
58 | 58 | alias for compatibility |
59 | 59 | * (bug 4525) Move section links down visually to same level as headings |
60 | 60 | editsection links are now inside the heading elements) |
| 61 | +* (bug 7623) Validate custom HTML id's correctly in Monobook interface |
61 | 62 | |
62 | | - |
63 | 63 | == Languages updated == |
64 | 64 | |
65 | 65 | * German (de) |