095 Home Stretch

7,075 Posts

davidm Reply #131, 19 years, 11 months ago

It’s not that urgent Garry (take a break, you’ve done a lot in a short span of time already ! ), I just wanted to know if it was identified or not... thanks for the quick answer

Have a nice sunday evening !

.: COO - Commerce Guys - Community Driven Innovation :.

MODx est l'outil id

MODX Staff
10,725 Posts

opengeek Reply #132, 19 years, 11 months ago

Quote from: garryn at Oct 08, 2006, 12:46 PM

I thought Garry’s change to DocManager were in Beta5, am I wrong here ?
The problems is because of the frame changes made recently ... I haven’t had chance to submit an update for that yet, will try and have a look tonight.

Whew...good news, I thought this was specific to my latest 097 branch changes. I’m off the hook on this one it seems though... wink

Jason Coward
Chief Architect @ MODX
http://www.jasoncoward.com | http://twitter.com/drumshaman | https://github.com/opengeek

680 Posts

aNoble Reply #133, 19 years, 11 months ago

Hey, sorry for being late to the party. I’m going to do my best to bring QuickEdit working as reliably as possible.

I’ve glanced over this thread and found a few things related to QE but I’m not sure what has been fixed alread and what is still an issue. In my own testing (with FF 1.5, Konqueror 3.5 and IE 6) the only obvious problem I’ve seen is that the drop-down-menus on the toolbar don’t work in Konqueror. I’ve already committed some CSS fixes for people like me who don’t have Arial installed and some fixes for the way the ContentVariable class checks if a document is richtext or not.

If you could all let me know what problems you’re still having with QE I’ll do my best to fix them. As many details, screenshots, source code, etc. you can give is appreciated.

Adam Crownoble: obleDesign.com | [email protected] | [email protected] (Google Talk)

2,204 Posts

Dimmy Reply #134, 19 years, 11 months ago

Quote from: aNoble at Oct 08, 2006, 06:19 PM

If you could all let me know what problems you’re still having with QE I’ll do my best to fix them. As many details, screenshots, source code, etc. you can give is appreciated.

here is one that needs fixing so that QE can be used by users that are "manager users" but have no manager access.

http://modxcms.com/bugs/task/545

follow me on twitter: @dimmy01

7,075 Posts

davidm Reply #135, 19 years, 11 months ago

Don’t think it’s a bug per say, but indeed this would be great grin

Is that in QuickEdit’s code or the authorization system though ?

.: COO - Commerce Guys - Community Driven Innovation :.

MODx est l'outil id

680 Posts

aNoble Reply #136, 19 years, 11 months ago

Quote from: Dimmy at Oct 09, 2006, 03:18 AM

here is one that needs fixing so that QE can be used by users that are "manager users" but have no manager access.

I’ll take a look at that but last time I checked that wasn’t really feasible. The problem is that QuickEdit is a plugin and a module the plugin (the toolbar) works fine because plugins don’t require manger access. But the module (the editor) is actually part of the manager. And unless we build in someway of overriding the "No manager access" setting in some cases it not possible to run QE.

If there’s no simple way to make this work I’ll at least disable the toolbar also when the user does not have manager access.

Adam Crownoble: obleDesign.com | [email protected] | [email protected] (Google Talk)

2,204 Posts

Dimmy Reply #137, 19 years, 11 months ago

QE is a great tool for noobs, so I realy hope that this works out.

there are just some people that you do not want to give manager access
another option would be to use a qe login instead of using /manager.
so when the user setting start id is set that the QE toolbar has no link to the manager. that way if noone tels the noob that there is a /manager it could work a bit.

does this make sense?

Dimmy

follow me on twitter: @dimmy01

680 Posts

aNoble Reply #138, 19 years, 11 months ago

Quote from: Dimmy at Oct 09, 2006, 07:34 AM

another option would be to use a qe login instead of using /manager.
so when the user setting start id is set that the QE toolbar has no link to the manager. that way if noone tels the noob that there is a /manager it could work a bit.

Normally what I do is use the ManagerLogin snippet so they can login from the frontend. Then you can hide the manager link in the QE toolbar in the QE module configuration options.

Adam Crownoble: obleDesign.com | [email protected] | [email protected] (Google Talk)

MODX Staff
12,272 Posts

Ryan Thrash Reply #139, 19 years, 11 months ago

Hey Garry, I think I found a minor bug in the range selection for the Doc Manager. I just tried the latest merged resources and was able to get the 21* syntax to work, but not the 21-27 syntax. Can you verify if that’s not just me? (This was for changing templates in Safari)

update: SVN rev 1655 FYI. I also was able to get a bulk change to work using: 21,22,23,24,25,26,26,27... The resulting error when using range selection is always:

Update has completed but encountered errors:

No selection was made.

Ryan Thrash, MODX Co-Founder
Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me

MODX Staff
1,535 Posts

Garry Nutting Reply #140, 19 years, 11 months ago

Hi Ryan, I’m aware of that issue and ~~will submit a fix for it tonight~~ the fix has now been submitted ... the fix (if you wanted to try it out) is to change returnval to $returnval on line 702 of the process.inc.php file.

Garry Nutting
Senior Developer
MODX, LLC

Email: [email protected]
Twitter: @garryn
Web: modx.com