As the title states, the 'cv-pls' button should be locked when a scheduled room meeting is happening. This stops the meeting from being interrupted (and the requests from being sent to the trash), so therefore effectively improves the focus of the meeting.

I tested a cv-pls on this meta post: Stop protecting [protect].

It didn't end well:

We need to either escape those values or strip them out, depending on which is easier or possible.

Script: Magic™ Editor
Issue type: Bug
Link to example post: (where can the problem be reliably replicated?)

This:

Generally i need to pick up `authData` and `RequestData` elements to the one level up (or to delete `processRequestRequest` element).


**UPDATE**

> Spring WS offers "only" contract-first, starting from an XSD Schema

...gets changed to this:

Generally I need to pick up `authData` and `RequestData` elements to the one level up (or to delete `processRequestRequest` element).> Spring WS offers "only" contract-first, starting from an XSD Schema

...it should be this:

Generally I need to pick up `authData` and `RequestData` elements to the one level up (or to delete `processRequestRequest` element).

> Spring WS offers "only" contract-first, starting from an XSD Schema

...but the white space between the Block-quote and the preceding sentence has been removed.

Observed in this edit, for example.

With App.globals.showRules = true;, isolated the problem to an apparent interaction between two rules, editupdate and blanklines, shown below. Debug info didn't show any unexpected replacement, so perhaps there is a third player? (Placeholder logic, maybe?)

With either rule disabled, there's no problem (but that the rule isn't doing its job).

        editupdate: {
            // https://regex101.com/r/tT2pK6/9
            expr: /([-_*]+[\t ]*\b(edit|update)\b([\t ]*#?[0-9]+)?[\t ]*:*[\t ]*[-_*]+:*|[\t ]*\b(edit|update)\b([\t ]*#?[0-9]+)?\s*:+[\t ]*)/gi,
            replacement: "",
            reason: App.consts.reasons.noise
        },

...
        blanklines: {  // https://regex101.com/r/eA5hA2/2
            expr: /^(?: *[\n\r\f])+|(?: *[\n\r\f])+$|((?: *[\n\r\f]){2})(?:(?: *[\n\r\f]))+/g,
            replacement: "$1",
            reason: App.consts.reasons.layout
        },

This might be related to issue #39 , in that the placeholders / block replacement logic could be involved.

(Migrated from old repository #115.)

I had a question the following title:

communication between JS, PHP and MySQL

and Magic Editor turned it into

Communication between JavaScript,

I think it should only remove tags from titles if they are in the beginning, not somewhere in the middle or at the end.

Another example:

Pass js values to php

was changed to:

Pass JavaScript values to

I don't yet have a solid test case but just now, after I edited a question and then tried to cast a close vote, I did not see a "send cv-pls request" checkbox in the close reason dialog. Reloading the page by clicking the question title and then reopening the close reason dialog brought it back.

There is a new version up. The new version contains the auto updater. If you could test it out for me by installing that version then change VERSION = '1.2' to VERSION = '1.1'. It should check once per day. If it says there is an update, try clicking cancel. Then select [Check for updates] from the menu. It should alert you of the update again. Then clicking [OK] should open a new tab to install the update. /Tiny

Performed the above test, starting with version 1.3. Performed multiple downgrade / upgrade cycles, and saw only one issue, the subject of this report.

• Installed v 1.3, was previously running a v 1.0 from Tiny's own gh repo. Uninstalled the old version.

• Loaded a question in another tab, validated that v1.3 was running by clicking cv-pls and observing the new 3-item menu.

• In GreaseMonkey, edited then saved user script.

  // @version      1.1
  ...
  var CURRENT = '1.1';
  

  ^{_{Left GM editor open, because by re-saving the edited version I could perform "downgrades" for retesting.}}

• Back to question, reloaded page. Was prompted for update, as expected. Cancelled.

• Clicked cv-pls and selected Check for updates. Was alerted of update, as expected.

• Updated, and confirmed update in GM.

• Back to question, reloaded page. Was prompted for update. <-- Suspected Bug

• Updated.

I have not been able to reproduce this again; I can downgrade & upgrade repeatedly now.

So, what happened? Theories...

• The "once per day" rule was in place for my first test, which explained the first notification when a question was opened, triggering the user script. Maybe cancelling that upgrade leaves an opening a the "once per day" check to run again.
• The first time I installed the new script, it didn't upgrade the previous version I had because it was from a different source. TM showed both as unique scripts, and I was able to remove the old one. Maybe there was some other interplay between them, perhaps via HTML5 storage.

Since the problem isn't persistently repetitive, it's a low impact issue.

Script: Magic™ Editor
Issue type: Bug
Link to example post: n/a

Users of SE like I like it find that the tab handling in ME conflicts with it.

Would be nice if tab handling was either optional or only took effect if there was no other competing script.

Reported in chat.

According to rene in chat here, SOCVR doesn't accept cv-pls / reopen-pls requests for Meta. Thus, I'd like to request that the CV Request Generator script be disabled on Meta sites.

When I wanted to change the default "SO close vote reviewers room" it says Invalid chat room. The room url is of course valid and works in Firefox.

When inspected, chrome shows a messages like this

refused to connect to "http://chat.stackexchange.com/rooms/0000/chat-room-name": URL is not part of the @connect list

I could fix the problem by just adding another line with chat.stachexchange.com domain.

(The room name I used here is a made-up name, but I used an active room. Even a room I owned)

It's a derogatory term and I'd rather not have that language in the chat room.

Right now the settings between SO and MSO are not linked. Should they be?

When we (or really I) push a new version of the script, users get the confirm box pushed in their face:

https://github.com/SO-Close-Vote-Reviewers/UserScripts/blob/master/SECloseVoteRequestGenerator.user.js#L86

Can we at least make this less blocking?

Reported several times in the past but recently by FerryBig: http://chat.stackoverflow.com/transcript/message/34171354#34171354

On Firefox, Ctrl-Shift-A is a stock shortcut for the Add-ons page about:addons (similar to chrome://extensions/ in Chrome).

Simple reproduction:

1. Click cv-pls action when viewing a question. Observe menu pop-up.
2. Abandon the action, by doing something else. For example, click share action. Observe that the pop-up menu remains visible.

I first noticed this by choosing to "Send Request", then hitting cancel in the resulting dialog.

Script: Magic™ Editor
Issue type: Enhancement
Link to example post: n/a

DavidG brought this up in chat.

See How can we avoid “overhang” code indentation?

Since the general message from that meta post is that automatic reformatting is difficult and bad, Magic Editor should not make magical changes to code. (Yay! Done that part already.)

However, providing the ability to farm out reformatting to a web service, under user control, might be really handy.

And DavidG wants a button. Maybe two or three. Make them big and shiny.

(Migrated from old repository #110.)

Script: Magic™ Editor
Issue type: Bug
Link to example post: See individual comments

Within the code, it's acknowledged that the magic that corrects the use of the indefinite articles "a" and "an" gets it wrong sometimes.

// Hack alert: Due to the technical nature of SO subjects, many common terms
// are not well-represented in the data used by AvsAnSimple, so we need to
// provide a way to override it.

If you come across an instance where the script got it wrong, please report it here, as a comment on the issue. Include the article chosen by the script and the word following the article, e.g. "an mistake". To do so, add a new comment to the thread.

NOTE: Because code is ignored by Magic™ Editor, indefinite articles appearing immediately before a back-ticked code snippet have no relevant context for evaluation, and are therefore always "a". Example:

You need to use an `IplWidget` object.

will become:

You need to use a `IplWidget` object.

(Migrated from old repository #114.)

Script: Close Vote Request Generator
Issue type: Bug
Link to example post: http://chat.stackoverflow.com/transcript/41570?m=29479725#29479725

Instead of showing the name of the post's author, the script populates the name of the author of the dupe-target question.

Script: Magic™ Editor
Issue type: Bug
Link to example post: http://stackoverflow.com/a/35704536/1677912 in original form

The "D" in "don't" is not correctly capitalized in this text:

i stumbled upon this post, looking to use css transform in jquery for an infinite loop animation. this one worked fine for me. dont know how professional it is though.

Which became:

I stumbled upon this post, looking to use CSS transforms in jQuery for an infinite loop animation. This one worked fine for me. don't know how professional it is though.

This seems to be the case for other contractions as well, e.g. "can't", "won't".

There is at least one outdated command in the BotCommands UserScript

On line #54, it lists room effectiveness as a command, which apparently doesn't work anymore.

Which makes no sense as yo can't close vote them anymore. Please remove the cv-pls link if the question is or gets deleted.

Reported by Fred -ii- in SOCVR Chat here

There are a couple changes I would suggest when setting a new target room.

1. The banner at the top doesn't stick around for long enough. There's no way I can read all that text by the time it goes away.

2. Let's change the banner text to show the name of the room rather than the url. Should be able to get the human-friendly name from the /rooms/info/# page.

3. After a new target room is set, the input elements are still visible. I think it should be collapsed back into the menu item once successfully changed.

4. Above the input box should read some text about what is currently set. Maybe something like

   The current target room for (site name) posts is (name of chat room).

Script: Magic™ Editor
Issue type: Enhancement
Link to example post: below

For example, a title like this:

How can I fix the issue

should become:

How can I fix the issue?

Anyway, just check if the title starts with "How", "When", "Why", "What", "Can", etc. Then put a question mark after it.

I can imagine durron wants this feature.

We should use html5 data storage for saving the chat room to post cv-plz messages to, saved per site you are on.

For example, if you are on Stack Overflow, it posts to room A, MSO posts to A as well, and MSE posts to room B.

If possible, use a dialog window similar to the flagging/closing interface of the site (same how the auto-comments work).

Alerts are bad, Mkay? A stupid user (hi) might just click "Prevent this page from creating additional dialogs”, thereby rendering the tool useless.

It is possible to check whether alerts are showing, to help stupid users (stop talking about me!) fix things.

Script: Magic™ Editor
Issue type: Enhancement
Link to example post: below

Identified by Rizier on this post, in its original form.

There is code surrounded by <pre> </pre> preformat HTML tags, which is not identified by MagicEdit as code.

Same for <code> </code>.

(Migrated from old repository #94.)

If you send a cv-pls request and don't refresh the page, the reason you used will still be filled in.

I'm actually not sure if this is a good idea or a bug. It shows you that you already requested it (in case you double-request by accident).

I just can't do it?!?

becomes

I just can't do It?!?

If the punctuation is just ? or !, then it is left alone. Weird.

Typically when I press the keystroke, I do the following:

1. Press Ctrl with my pinky
2. Press Shift with my ring finger
3. Press a with my middle finger
4. Lift my pinky
5. Lift my ring
6. Lift my middle finger

If I do this too quickly, the cv-pls dialog does not appear. However, if instead I:

1. Lift my middle finger

It seems to work. This bug can be reproduced always on Chrome 44.0.2403.155 m, with script version 1.4.6.

Script: Magic™ Editor
Issue type: Bug
Link to example post: How to setup login method to avoid duplicate code every test [selenium][java]

The title of this post is: How to setup login method to avoid duplicate code every test [selenium][java]

MagicEditor does not remove the two tags from this title. Tried a new regexp for the taglist rule, but it doesn't completely remove the tags + braces. (Leaves How to setup login method to avoid duplicate code every test [.

Personally, I like Ctrl+Shift+a, but others may not, especially in browsers not Chrome.

Can we have the ability to customize it in the cv-plz menu?

Script: Magic™ Editor
Issue type: Feature Request
Link to example post: n/a

It would be nice if the script recognized changes in the tags, and inserted a comment for them if one wasn't already present. ("Edited tags; ")

(Migrated from previous repository #75).

Script: Magic™ Editor
Issue type: Enhancement
Link to example post: below

For proper layout in lists, it's necessary to indent code cod by multiples of 4 spaces.

Example post: The original version of this question.

1. Code Service Provider:

   class EloquentEventServiceProvider extends ServiceProvider
   {
   public function boot()
   {}

   public function register()
   {
       $this->app->bind('app\\Repositories\\EventRepository', 'app\\Repositories\\Eloquent\\EventEloquentRepository');
   }
   

   }

The code here has been indented 4 spaces by use of the "code" button in the default editor, however it needs to be indented 8 characters because it's within a numbered list.

It would be nice if Magic™ Editor let us select the code block and click a button to shift it right 4 more spaces.

(Migrated from old repository #92.)

The request at https://chat.stackoverflow.com/transcript/message/37904956#37904956 is from user8234819, with no link. checkDone() looks for the reporting-user's <a> tag and blindly tests its href attribute. Since there is no link here, we fail with

Uncaught TypeError: Cannot read property 'href' of null
at Object.funcs.checkDone (eval at E_c (:3:221), :82:83)
at XMLHttpRequest.xhr.addEventListener.event (eval at E_c (:3:221), :218:27)

Script: Magic™ Editor
Issue type: Bug
Link to example post: https://stackoverflow.com/posts/34326658/edit/4979dcb3-38f5-410b-9501-4af81ac2e023

string.replace() might help

becomes

string.

(Migrated from old repository #96.)

I use Chrome. I leave the chat room on one monitor and the post I'm working on the other.

Let's say I want to change the room I'm posting to.

1. Go to the post.
2. Hit cv-pls menu and choose "Set target room"
3. Go to the chat room and copy the url

Oh wait, I can't. window.prompt blocks user input to the entire browser until it's finished. Same with alert.

It's frustrating :(

Here's an example dialog box I whipped up real fast.

var postMenu = $('#question .postcell .post-menu');

var popupBox = $('<div class="popup" style="position: absolute; top: 399.5px; left: calc(50% - 416px); display: block;"></div>');
popupBox.append('<div class="popup-close"><a title="close this popup (or hit Esc)">×</a></div>');
popupBox.append('<h2>This is the popup window title!</h2>');

//add more things here

postMenu.append(popupBox);

If we can get this working, we can make the "request sent successfully" auto-disappear, so you don't have to spend a mouse click to make it go away.

Script: Magic™ Editor
Issue type: Bug
Link to example post: below

I encountered a question in the Suggested Edit queue, and chose to Improve Edit. After clicking the auto-edit button, the diff included one of our _xTagxPlacexHolderx_ strings in the deleted text, where a <br> had been in the original text. Just confusing.

Link to edit: http://stackoverflow.com/review/suggested-edits/10359835

Link to revision being edited: http://stackoverflow.com/revisions/33958915/2

(Migrated from old repository #77.)

Could you move the help from the SOCloseVote script into the README.md from the comments in the user script?

Thanks!

Script: Magic™ Editor
Issue type: Enhancement
Link to example post: n/a

http://www.oxforddictionaries.com/words/common-misspellings

I looked at just two of these, "across" and "immediately", and implemented rules that matched >8000 posts.

        across: { 
            expr: /\b(a)c+ros+\b/gi,
            replacement: "$1cross",
            reason: App.consts.reasons.spelling
        },
        immediate: {
            expr: /\b(i)m+ed[ia]+te?l?(ly)?\b/gi,
            replacement: "$1mmediate$2",
            reason: App.consts.reasons.spelling
        },

It would be a useful exercise to work through the rest of that list.

(Migrated from old repository #113.)

So this was a bit strange:

It seems that if the text from an input box goes off the side of the box, and you try to click and select text (like if you wanted to go to the end of the string to see what room is set), it will freak out and start inserting javascript:void(0) in the text.

I can still use the arrow keys, end key, and select + a, but I would still like to be able to select text on my own.

I often use the keyboard shortcuts to quickly cast close votes, but the "Send CV request" check box does not appear when I use them.

Script: Magic™ Editor
Issue type: Bug
Link to example post: Any post in suggested edits

When improving edits in the Suggested Edits Reviews queue, the Tab key changes fields. This is expected to indent code. Likewise with shift-tab, which should outdent; that does not work in the queue.

TBD: Is this limited to the Suggested Edits, or does it happen in other queues?

It's sometimes really hard to click a menu item. The hitboxes are just too small compared to their container.

The real killer is that the entire clickable region changes the cursor to a pointer, so the only indication that you are on the right part is the link changes color.

Script: Magic™ Editor
Issue type: Auto-fix term request
Link to example post: https://regex101.com/r/pN9fY6/3

There are more than 43,000 posts containing a variant of this phrase, "Does anyone know how...". That is a colloquial term for "how", but it is literally a different question, so it is not clear. (Do people really want to know if someone knows something,? No, they are asking "how?")

As shown in the linked RegEx101, it's not a straight-forward replacement, except in a subset of cases. It would probably require a two-stage replacement, ala firstcaps.

Script: Magic™ Editor
Issue type: Enhancement
Link to example post: http://stackoverflow.com/revisions/22688660/1

Markdown expects numbered lists to use the format n. to start list items.

Many users instead use n), which is not recognized. Other users neglect to leave a blank line (double space) before the first list item (even if it's formatted correctly otherwise), which also negates formatting.

This:

Blah blah blah
1) blah blah blah
2) blah blah blah

...should become this:

Blah blah blah

1. blah blah blah
2. blah blah blah

A blank line between items is optional.

I'm constantly nagged about cross-origin resource requests with the SOCVR Request Generator script, whenever I do anything on the page besides from loading it, for example:

• Refreshing;
• Voting;
• Flagging;
• Commenting

Please could this be fixed?
I've now set all requests to 'ignore', but now I'm experiencing an Failed retrieving fkey from chat notification message where the 'Sent cv-pls request' message should be.

Script: Magic™ Editor
Issue type: Bug
Link to example post: everywhere

Diff is showing ↵ in Firefox and Chrome, typically near the end of posts.

Happens when I'm attempting to send a 'cv-pls' to SO CV Reviewers Chat.
On Chrome 44.0.2403.157 m on Windows 8.1 Professional.

I tried to read the code of Magic Editor and looks like some part are minified. Is it possible to have a non minified thing ?

The question tag (most relevant or all) should be added as es. [java] not [tag:java] to the cv-pls request, before or after the title.

Reasons:

1. On duplicate request, hammer in SOCVR chat can immediately identify if he can hammer.
2. User in chat and when searching on previous requests can filter on specific tag.

Other than looking at chat, there is no indication that the process has completed. I suggest that we add a small dialog window that says "cv-plz posted successfully" which fades away after 1.5-2 seconds.