Обсуждение MediaWiki:Gadget-common-special-upload.js

message: Время от времени появляются багрепорты о том, что не сабмитится форма. Я уже много раз перечитывал код, но ошибок там не нашёл. Видимо, их там и нет. Но отсутствие ошибок не обязательно означает, что всё будет работать так, как планировалось. Поэтому предлагаю внести следующие замены в код, есть шанс, что они помогут.
function uploadPage(){
 if (wgUploadAutoFill){
  upForm = document.getElementById('mw-upload-form')
  upAct = upForm.action
  upForm.action = 'javascript:checkUpForm("'+upAct+'");void(0);' // заменено (1)
 }
 ...
}
function checkUpForm(upAct){ // заменено (2)
 upForm = document.getElementById('mw-upload-form') // добавлено (3)
 upForm.action = upAct
 var desc = document.getElementById('wpUploadDescription')
 var lcns = document.getElementById('wpLicense')
 var wrn = document.createElement('td')
 wrn.innerHTML =  // добавлено (4)
 if (...)
  wrn.innerHTML += '<li>...</li>'
 ...
 if (...)
  wrn.innerHTML += '<li>...</li>'
 if (wrn.innerHTML != ){
  var wrnPrnt = document.createElement('tr')
  wrnPrnt.appendChild(document.createElement('td'))
  wrnPrnt.appendChild(wrn)
  wrn.style.border = '1px solid red'
  wrn.style.background = '#ffe4e1'
  wrn.innerHTML = '<p style="text-align:center; font-weight:bold;">Автоматическая проверка выявила следующие ошибки:</p><ul style="font-weight:bold">' + wrn.innerHTML
  wrn.innerHTML += '</ul><p style="padding:0.2em 0.8em">При повторном нажатии на кнопку «Загрузить файл» данное предупреждение будет проигнорировано. Помните, что файлы без лицензии или с некорректной лицензией будут удалены. Если у вас возникли проблемы или вопросы, обращайтесь на <a href="http://ru.wikipedia.org/wiki/%D0%92%D0%B8%D0%BA%D0%B8%D0%BF%D0%B5%D0%B4%D0%B8%D1%8F:%D0%A4%D0%BE%D1%80%D1%83%D0%BC/%D0%90%D0%B2%D1%82%D0%BE%D1%80%D1%81%D0%BA%D0%BE%D0%B5_%D0%BF%D1%80%D0%B0%D0%B2%D0%BE">форум по авторскому праву</a>.</p>'
  var upNode = document.getElementsByName('wpUpload')[0].parentNode.parentNode
  upNode.parentNode.insertBefore(wrnPrnt, upNode)
 }
 else upForm.submit()
}
* (1) и (2) — самая вероятная ошибка; если при сабмите форма пересылает на себя же, то скорее всего action='', т. е. upAct по какой-то причине обнулился;
* (3) — другая вариация той же ошибки, когда кнопка вообще не реагирует, т. е. не возвращается действие по умолчанию;
* (4) — innerHTML для нового элемента вроде как должен быть пустым, но я уже сомневаюсь. Впрочем, это можно пока не трогать.
Прошу кого-нибудь проверить и внести изменения. — putnik 08:19, 9 июля 2009 (UTC) upd: 09:02, 9 июля 2009 (UTC)
** Протестировал. Вроде бы всё отлично работает. — putnik 20:08, 9 июля 2009 (UTC)
*** Исправлено (спасибо Лемме). — putnik 10:50, 22 июля 2009 (UTC)
thread_title:  Проблемы с сабмитом формы 

message: В связи с заменой схемы надо бы обновить. Но как грузят сейчас, я вообще не представляю, ибо он ругается на новую схему. Я на днях обдумаю, но пока можно заменить на такие:
 if (lcns.value ==  && /^[^\{\}]*(\{\{(Изображение|Несвободный файл|Несвободный файл\/ОДИ|Обоснование добросовестного использования|ОДИ)[^\{\}]+\}\}[^\{\}]*)*$/i.exec(desc.value))
  wrn.innerHTML += '<li>Не указана лицензия. Выберите лицензию в выпадающем списке выше.</li>'
 if (lcns.value !=  && /\{\{fair ?use ?in *(\|[^\{\}]*)? *\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>Лицензия указана и в описании, и в выпадающем списке. Возможно, вы где-то ошиблись.</li>'
 if (/\{\{fair ?use ?in *\|? *\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>В шаблоне {'+'{fairusein}} не указано название статьи.</li>'
 if (/\{\{fair ?use ?in *\| *название статьи *\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>В шаблоне {'+'{fairusein}} текст «название статьи» не изменён на реальное название.</li>'
 if (lcns.value ==  && /\{\{Изображение\s*((\|\s*)+[\s\wа-яё]+=\s*)*(\|\s*)*\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>Не заполнен шаблон {'+'{Изображение}}.</li>'
 if (lcns.value ==  && /\{\{(Обоснование добросовестного использования|ОДИ)\s*((\|\s*)+([\wа-яё]+\s*)+=\s*)*(\|\s*)*\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>Не заполнен шаблон {'+'{Обоснование добросовестного использования}}.</li>'
 if (lcns.value ==  && /\{\{Несвободный файл\s*((\|\s*)+([\wа-яё]+\s*)+=\s*)*(\|\s*)*\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>Не заполнен шаблон {'+'{Несвободный файл}}.</li>'
 if (lcns.value ==  && /\{\{Несвободный файл\/ОДИ\s*((\|\s*)+([\wа-яё]+\s*)+=\s*)*(\|\s*)*\}\}/i.exec(desc.value))
  wrn.innerHTML += '<li>Не заполнен шаблон {'+'{Несвободный файл/ОДИ}}.</li>'
 if (/\{\{(Обоснование добросовестного использования|Несвободный файл|ОДИ|Изображение)[^\{\}]*Источник\s*=\s*(из|с|http:\/\/(www\.)?)?\s*(интернет|internet|гугл|google|яндекс|yandex|яху|yahoo|рамблер|rambler)[^\{\}]*\}\}/i.test(desc.value))
  wrn.innerHTML += '<li>«Интернет» или поисковая система не являются источником, подтверждающим лицензионный статус изображения. Укажите более конкретный источник.</li>'
— putnik 08:28, 9 июля 2009 (UTC)
thread_title:  Правила проверки 

message: MediaWiki developers found that this page probably breaks JavaScript for users (example: not seeing the buttons when editing a page). You probably need to edit this .js page and/or MediaWiki:Gadgets-definition as in the examples at phabricator:T122755. [/w/index.php?search=insource%3A%2F(mw.util|mw.notify)%2F+-loader.using&title=Special:Search&fulltext=1&ns8=1 List more pages to check].
If you have questions or need help, please ask at phabricator:T164242. You can login with your wiki account. Best wishes, Nemo 09:49, 14 мая 2017 (UTC)
thread_title:  Broken JavaScript