? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
22 Jul 2016

Pull Request for New Issue.

Summary of Changes

Simple PR.
Categories table doesn't allow to customize the category creator on new category.
Content table allows just that (in https://github.com/joomla/joomla-cms/blob/staging/libraries/legacy/table/content.php#L323-L333).

This PR change the categories table to allow that too.

Testing Instructions

simple test

  • Use latest staging
  • Besides "Super User" create and additional user "Test User"
  • Create a new category and change the category creator to be the "Test User" before saving image
  • Save the category, edit it again and check the Creted By is "Super User" instead of "Test User"
  • Apply patch
  • Repeat steps. All good now

Note: This will also allow to do this in bulk import trough JTable

avatar andrepereiradasilva andrepereiradasilva - open - 22 Jul 2016
avatar andrepereiradasilva andrepereiradasilva - change - 22 Jul 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jul 2016
Category Libraries
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jul 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Jul 2016
avatar brianteeman
brianteeman - comment - 22 Jul 2016

Why would you want to. What is the use case for this? The category creator
is never displayed. Is it?

avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Jul 2016

See the picture

avatar brianteeman
brianteeman - comment - 22 Jul 2016

I did. My question remains. When do we ever show/use the category creator

avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Jul 2016

Well...

  • This corrects a bug as i describe above
  • Also if a field apperars or not it depends of the template (protostar i think don't show, but other template can show, or even in metadata). The point is, if you have a field in the database it can be used, and so it needs to be consistent.
  • Anyway my main goal is to correct the creator of the category in multilanguage install (bulk create) and i need this bug corrected for that
avatar brianteeman
brianteeman - comment - 22 Jul 2016

Your final point is the use case I was looking for :)

avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Jul 2016

?

avatar andrepereiradasilva
andrepereiradasilva - comment - 22 Jul 2016

closed. incorporated in #11252

avatar andrepereiradasilva andrepereiradasilva - change - 22 Jul 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-07-22 22:07:48
Closed_By andrepereiradasilva
avatar andrepereiradasilva andrepereiradasilva - close - 22 Jul 2016

Add a Comment

Login with GitHub to post a comment