Исчезающие команды меню после ввода пользователя в Tampermonkey - PullRequest
1 голос
/ 13 марта 2011

Tampermonkey - это расширение для Google Chrome, которое пытается эмулировать функциональность Greasemonkey.Чтобы было ясно, я заставил свой скрипт работать в Chrome, и JavaScript по умолчанию изменился, чтобы показать его.Однако я хотел проверить команды меню и ввел 6-значный шестнадцатеричный цветовой код после нажатия на команду в меню Tampermonkey.Я перезагрузил страницу, и команды исчезли из меню!Мой сценарий был все еще там (и флажок был отмечен).

Независимо от того, что я сделал или какой код изменил, я никогда не смогу эмулировать эту начальную функциональность после того, как был задан этот пользовательский ввод.Это заставляет меня поверить, что есть некоторые постоянные данные, которые я не могу удалить, что приводит к преждевременному сбою сценария.ПРИМЕЧАНИЕ. Этот точный скрипт работает без ошибок в Firefox.

Это явно не форум Tampermonkey, но люди здесь, кажется, очень хорошо осведомлены о кроссплатформенной совместимости.Я не слышал ни единого взгляда с консоли Chrome после всех приведенных ниже изменений, и на данный момент у меня просто нет идей.Вот некоторые вещи, которые я пробовал (без успеха).Перечислены все ошибки консоли:

  1. Изменение версии jQuery с 1.5.1 на 1.3.2
  2. Вызов localStorage.getItem ('prevoColor') из консоли после загрузки страницы (оба значения равны нулю).)
  3. Изменение хранилища на стороне клиента из localStorage для получения / setValue
  4. Вызов GM_getValue из консоли = ReferenceError: GM_getValue не определено
  5. Удаление записей localStorage для veekun.com вПараметры Chrome
  6. Обновление, переустановка сценария и перезапуск браузера больше, чем я могу сосчитать
  7. Повтор всех вышеперечисленных команд с использованием Firebug Lite (букмарклет)

Вот код, который я использовал:

// ==UserScript==
// @name           Veekun Comparison Highlighter
// @namespace      tag://veekun
// @description    Highlights moves exclusive to pre-evolutions on veekun.com's family comparison pages (user-defined colors available)
// @include        http://veekun.com/dex/gadgets/*
// @author         Matthew Ammann
// @version        1.0.3
// @date           3/11/11
// @require        http://sizzlemctwizzle.com/updater.php?id=98824
// @require        http://ajax.googleapis.com/ajax/libs/jquery/1.3.2/jquery.min.js
// ==/UserScript==

/*
Goal: Change checkmark color & move name to user-specified color on family comparison pages if 
[DONE] Baby poke has a LEVEL-UP move unlearned by any evolutions
    [DONE] a) Make sure move is not a TM or tutor move
[DONE] Any other mid-evolution has a move unlearnable by a final evo (Caterpie, Weedle families)
    [DONE] a) Make sure move is not a TM or tutor move
[DONE] Any pre-evo has a TUTOR move unlearned by any evo (Murkrow in HG/SS)
[] Implement auto-update after uploading to userscripts.org

Credits: Brock Adams, for helping with Chrome compatibility
         Metalkid, for the jQuery consult
*/

var isLevelupMove = false;
var isTutorMove = false;
var isTM = false;
var TMhead = $('#moves\\:machine');
var hasSecondEvo = false;
var hasFinalEvo1 = false;
var hasFinalEvo2 = false;
var header = $('.header-row').eq(1);
var TMmoves = new Array();

//This section deals with the user-defined colors 

GM_registerMenuCommand("Color for pre-evolutionary-only moves", prevoColorPrompt)
GM_registerMenuCommand("Color for first evolution-only moves", evoColorPrompt)


var prevoColor = GM_getValue('prevoColor', '#FF0000');
var evoColor = GM_getValue('evoColor', '#339900');

function prevoColorPrompt()
{
    var input = prompt("Please enter a desired 6-digit hex color-code for pre-evolutionary pokemon:") 
    GM_setValue('prevoColor', '#'+input);
}

function evoColorPrompt()
{
    var input = prompt("Please enter the desired 6-digit hex color-code for first-evolution pokemon:") 
    GM_setValue('evoColor', '#'+input);
}

//This loop tests each 'th' element in a sample header row, determining how many Evos are currently present in the chart.

$('.header-row').eq(1).find('th').each(function(index) 
{
    if($(this).find('a').length != 0)
    {
        switch(index)
        {
            case 2:
            hasSecondEvo = true;
            break;

            case 3:
            hasFinalEvo1 = true;
            break;

            case 4:
            hasFinalEvo2 = true;
            break;
        }
    }
});

//All 'tr' siblings are TM moves, since it's the last section on the page
//This array puts only the names of the available TMs into the TMmoves array
TMhead.nextAll().each(function(index)
{
    TMmoves.push($(this).children(":first").find('a').eq(0).html());
});

$('tr').each(function(index) 
{
    var moveName = $(this).children(":first").find('a').eq(0).html();   
    moveName = $.trim(moveName);

    switch($(this).attr('id'))
    {
        case 'moves:level-up':
            isLevelupMove = true;   
            break;

        case 'moves:egg':
            isLevelupMove = false;
            break;  

        case 'moves:tutor':
            isTutorMove = true;

        case 'moves:machine':
            isTM = true;    
    }

    if(isLevelupMove || isTutorMove)
    {
        var babyMoveCell = $(this).find('td').eq(0);
        babyMoveText = $.trim(babyMoveCell.html());

        secondEvoCell = babyMoveCell.next();
        secondEvoText = $.trim(secondEvoCell.html());

        finalEvo1Cell = secondEvoCell.next();
        finalEvo1Text = $.trim(finalEvo1Cell.html());

        finalEvo2Cell = finalEvo1Cell.next();
        finalEvo2Text = $.trim(finalEvo2Cell.html());

        //This checks if evolutions have checkmarks

        if(babyMoveText.length > 0)
        {
            if(hasSecondEvo && secondEvoText.length == 0 || hasFinalEvo1 && finalEvo1Text.length == 0 || 
                hasFinalEvo2 && finalEvo2Text.length == 0)
            {
                //See if the move is a TM before proceeding 
                var tm = tmCheck(moveName);

                if(!tm)
                {

                    if(secondEvoText.length > 0)
                    {       
                        babyMoveCell.css("color", evoColor);
                        secondEvoCell.css("color", evoColor);
                        babyMoveCell.prev().find('a').eq(0).css("color", evoColor); //highlights move name
                    }
                    else
                    {
                        babyMoveCell.css("color", prevoColor);
                        babyMoveCell.prev().find('a').eq(0).css("color", prevoColor);
                    }
                }
            }
        }
        else if(secondEvoText.length > 0)
        {
            if(hasFinalEvo1 && finalEvo1Text.length == 0 || hasFinalEvo2 && finalEvo2Text.length == 0)
            {
                var tm = tmCheck(moveName); 

                if(!tm)
                {
                    secondEvoCell.css("color", evoColor);
                    babyMoveCell.prev().find('a').eq(0).css("color", evoColor);
                }
            }
        }
    }

});

function tmCheck(input)
{
    var isTM = false;

    //Iterate through TMmoves array to see if the input matches any entries
    for(var i = 0; i < TMmoves.length; i++)
    {   
        if(input == TMmoves[i])
        {
            isTM = true;
            break;
        }
    }

    if(isTM == true)
        return true;
    else
        return false;       
}

//alert("evoColor: " + localStorage.getItem('evoColor') + ". prevoColor: " + localStorage.getItem('prevoColor'));

Любые идеи относительно того, почему это происходит?

EDIT : я отправил sizzlemctwizzle об этой проблеме, иэто был его ответ: «Реализация @require Tampermonkey неверна. Он загружает мой обновитель слишком часто, поэтому я запретил ему использовать мой обновитель через анализатор браузера.rver просто не может справиться с трафиком, который он приносит.Скрипт, который он загружает с моего сервера, не должен содержать никакого реального кода.Так как это вызывает ошибки в вашем скрипте, я бы предположил, что Tampermonkey не передает заголовок User Agent при выполнении этих запросов.Я никогда не проверял свой апдейтер в Chrome, поэтому понятия не имею, почему он ломается.Возможно, вы могли бы попробовать установить NinjaKit вместо этого. "

1 Ответ

1 голос
/ 14 марта 2011

По какому URL вы это тестируете?Я тестировал на http://veekun.com/dex/gadgets/stat_calculator.

В любом случае, поведение скрипта в отношении команд меню казалось неправильным с Tampermonkey.Я не мог сказать / действительно не проверял, работает ли остальная часть сценария так, как и должно.

Кажется, виновником является проверка обновления sizzlemctwizzle.com.Удаление // @require сделало меню стабильным.Отложив эту директиву обратно, снова сломал скрипт.

Я никогда не был фанатом этого средства проверки обновлений, поэтому я не буду вдаваться в подробности, почему он, кажется, нарушает экземпляр Tampermonkey этого скрипта.(Это был бы другой вопрос, и один из них, вероятно, лучше всего адресовать двум ответственным разработчикам.)

А пока предлагаем просто удалить его.Ваши пользователи будут проверять наличие обновлений по мере необходимости :).

...