Skip to:

Opened 9 years ago

Closed 8 years ago

#625 closed defect (fixed)

Plugin activation/ wponce troubles

Reported by: so1o Owned by: mdawaffe
Milestone: 0.8.2 Priority: normal
Severity: normal Version: 0.7.5
Component: Back-end Keywords: has-patch


the plugin page activation/deactivation does not work if the plugin is in a subdirectory...

Attachments (2)

Ticket-625.2.patch (962 bytes) - added by so1o 9 years ago.
patch for plugin and theme fix
Ticket-625.patch (1.4 KB) - added by so1o 9 years ago.
patch for plugin and theme fix

Download all attachments as: .zip

Change History (30)

#1 @mdawaffe
9 years ago

  • Owner set to mdawaffe
  • Status changed from new to assigned

Works fine for me.

Is it just one subdirectory deep in the hierarchy?

Are you running this on Windows? I thought the new code was meant to avoid all Unix v. Windows slash issues, but... :)

#2 @so1o
9 years ago

its just one sub dir..


. what it does is asks for confirmation recurssively.

im running it on windows.. also one more thing.. everytime its adds more slashes to the dir so the first time it asks for confirmation the link shows


the next time its


#3 @mdawaffe
9 years ago

(In [789]) Fix? for plugin activation on windows. re #625

#4 @mdawaffe
9 years ago

so1o, can you try the above fix and see if it solves the problem?

#5 @so1o
9 years ago

no .. but interestingly this works...

$plugin = stripslashes(stripslashes(trim($_GET['plugin'])));

#6 @so1o
9 years ago

infact i just noticed im having this issue with the themes page also.. i think its the same slashes issue.

#7 @so1o
9 years ago

ok.. so here's the deal..

in many places we do this -

bb_nonce_url( add_query_arg( ..

this is causing the problem.

add_query_arg function converts \ to
. and internally bb_nonce_url calls add_query_arg so at the end of it we have four \s for each \.

michael.. thoughts?

#8 @mdawaffe
9 years ago

Are you running the WordPress script at the same time? If so, what version?

#9 @so1o
9 years ago

the installation is not integrated.

i had put in some debugging comments in the code.. it runs the add_query_arg from the wp_functions file.

just a sidenot we should change the calls to add_query_arg to bb_add_query_arg

#10 @so1o
9 years ago

duh.. please ignore my last 'sidenote'.. i was off my medication when i wrote the 'sidenote'

#11 @mdawaffe
9 years ago

so1o, what do the following functions return?


#12 @mdawaffe
9 years ago

Also, what do you get if you


#13 @so1o
9 years ago







array(3) {
  string(13) "anonymous.php"
  string(37) "forum-moderators\forum-moderators.php"
  string(9) "roles.php"

#14 @mdawaffe
9 years ago

(In [802]) add_query_arg() to [WP5193] re #625

#15 @mdawaffe
9 years ago

so1o, can you try that latest update to see if it works?

If it does, does it still work if you take out [789] (the stripslashes)?

#16 @so1o
9 years ago

actually it doesnt.. i'll try a couple of stunts of my own here..

#17 @so1o
9 years ago


dangit! it was a dud!

here is the patch!

#18 @so1o
9 years ago

  • Keywords has-patch added

9 years ago

patch for plugin and theme fix

#19 @so1o
9 years ago

had the same problem in themes.. updated the patch

#20 @so1o
9 years ago

ok the final patch for this.. i promise!

9 years ago

patch for plugin and theme fix

#21 @mdawaffe
9 years ago

I think this is all a result of a couple inconsistencies in add_query_arg()

#WP4084 and #WP4105

I'm going to wait until the dust settles on them.

#22 @so1o
9 years ago

sure thing.

#23 @so1o
9 years ago

this also messes up the pagination

the url comes out as forum.php?id=1&page=%25_%25 and is forwarded to the paginate_links

which will not work.

#24 @mdawaffe
9 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [808]) add_query_arg() and urlencode_deep() from WP. Fixes #625

#25 @mdawaffe
9 years ago


Can you verify this works on Windows too?

#26 @so1o
8 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Nope not working! investigation underway...

#27 @mdawaffe
8 years ago

  • Milestone changed from 1.0 to 0.8.2

#28 @mdawaffe
8 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

Think it was the same thing as #558

Marking as fixed.

Note: See TracTickets for help on using tickets.