Révision b6ba57f6
Ajouté par Phil Davis il y a environ 10 ans
| usr/local/www/diag_tables.php | ||
|---|---|---|
| 106 | 106 |
window.location='diag_tables.php?type=' + entrytype; |
| 107 | 107 |
} |
| 108 | 108 |
function del_entry(entry) {
|
| 109 |
jQuery.ajax("diag_tables.php?type=<?php echo htmlspecialchars($tablename);?>&delete=" + entry, {
|
|
| 109 |
jQuery.ajax("diag_tables.php?type=<?php echo htmlspecialchars($tablename);?>&delete=" + entry, {
|
|
| 110 | 110 |
complete: function(response) {
|
| 111 | 111 |
if (200 == response.status) {
|
| 112 | 112 |
// Escape all dots to not confuse jQuery selectors |
| ... | ... | |
| 154 | 154 |
</tr> |
| 155 | 155 |
<?php $count = 0; foreach($entries as $entryA): ?> |
| 156 | 156 |
<?php $entry = trim($entryA); ?> |
| 157 |
<tr id='table_row_<?=$count?>'>
|
|
| 157 |
<tr id='<?=$entry?>'>
|
|
| 158 | 158 |
<td> |
| 159 | 159 |
<?php echo $entry; ?> |
| 160 | 160 |
</td> |
Formats disponibles : Unified diff
Fix Bug #3627 Diagnostics: Tables - Remove button dont work after update to PfSense 2.1.2
This annoyed me also, so I thought it worth finding what changes exactly broke this.
del_entry was broken on 2.1 branch by https://github.com/pfsense/pfsense/commit/fe3088b965a99772e76622d17ceae87288471edc
These 2 small changes make it work again without needing to reverse the other bits of stuff in that commit.
Note that Master does not have that commit at all. There are lots of these "XHTML Compliance" and similar commits in 2.1 branch that are not done in master. I don't understand why that is. Why don't those code cleanups also need to be applied to master?