Montego Scripts: Forums
 

 

View next topic
View previous topic
Post new topic   Reply to topic
Author Message
Guardian
Site Admin
Site Admin



Joined: Jul 18, 2005
Posts: 397

PostPosted: Tue Dec 13, 2011 3:35 pm Reply with quote

I had just dropped a module I was working on into my production site for further testing and realised something was a bit hinky.
I had made a bit of a no-no and forgot to prefix some of my admin functions and when I thought about it, I thought I had better mention it here in case someone else accidentally does the same thing due to the name of the function.
In the latest and greatest TegoNuke Downloads (sorry forgot the official title) one of the admin functions and CASE switches is Category Smile
 
View user's profile Send private message
montego
Site Admin/Owner
Site Admin/Owner



Joined: Feb 12, 2005
Posts: 1399

PostPosted: Wed Dec 14, 2011 9:41 pm Reply with quote

Oh my, that is ugly. Way too generic. Ok, thanks for letting me know. I'll add it to my issue tracker for a future enhancement release to change this.

_________________
“To err is human, but when the eraser wears out ahead of the pencil, you’re overdoing it.”
-- Josh Jenkins 
View user's profile Send private message Visit poster's website
montego
Site Admin/Owner
Site Admin/Owner



Joined: Feb 12, 2005
Posts: 1399

PostPosted: Thu Dec 15, 2011 8:52 am Reply with quote

I have captured an enhancement request for admin case collisions.

_________________
“To err is human, but when the eraser wears out ahead of the pencil, you’re overdoing it.”
-- Josh Jenkins 
View user's profile Send private message Visit poster's website
Guardian
Site Admin
Site Admin



Joined: Jul 18, 2005
Posts: 397

PostPosted: Thu Dec 15, 2011 9:29 am Reply with quote

Nice one!
I tend to over concentrate on things I feel are important, like accessibility and then sometimes completely forget to check obvious things. I just found two of my modules use AddCat doh!
I guess I have some work to do now Sad

Incidentally, I fully agree with your assessment there.
It would be really nice if admin.php could be used just as a router, leaving modules/index.php to be the handler instead of admin.php trying to do it all.
It needs to be like modules.php so you can route things with admin.php?module=MODULE_NAME&op..........

Luckily, there are less than a handful of people writing stuff for RN so I guess the risks are minimal.
 
View user's profile Send private message
Display posts from previous:       
Post new topic   Reply to topic

View next topic
View previous topic
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


Powered by phpBB © 2001-2007 phpBB Group
All times are GMT - 4 Hours
 
Forums ©
linear-bunchlinear-bunch