? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
15 Nov 2014

Executive summary

This PR converts the form validation on com_newsfeed to use plain jquery (no mootools call on every form).
Also NO MORE INLINE SCRIPTS!

Testing

  1. Apply first PR #4888 (!important)
  2. Apply this PR
  3. In the admin area create new menu items for newsfeed category
  4. go to the new menu and test that everything (the list reordering) is still OK

If no javascript errors are logged in your browser’s console and the functionality remains the same, your test is passing in any other case please report the errors here

Please also check these:
administrator/index.php?option=com_checkin should demonstrate multiselect without mt
administrator/index.php?option=com_users&view=mail should demonstrate form sent and validate without mt
administrator/index.php?option=com_modules should demonstrate multiselect and combobox without mt
Logout and log in to demonstrate the use of noframes without mt.

avatar dgt41 dgt41 - open - 15 Nov 2014
avatar jissues-bot jissues-bot - change - 15 Nov 2014
Labels Added: ?
avatar zero-24 zero-24 - change - 16 Nov 2014
Category Front End JavaScript
34705e4 17 Nov 2014 avatar dgt41 init
avatar anibalsanchez
anibalsanchez - comment - 17 Nov 2014

@test success. It is just a minor code change. Front-end site works as usual.

avatar roland-d roland-d - alter_testresult - 19 Nov 2014 - anibalsanchez: Tested successfully
avatar roland-d
roland-d - comment - 19 Nov 2014

@smanzi Would you mind testing this one also? Thank you.

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5115.

avatar dgt41
dgt41 - comment - 3 Dec 2014

@smanzi Sergio can you test this one?

avatar smanzi
smanzi - comment - 3 Dec 2014

yeap... hold-on...

avatar smanzi
smanzi - comment - 3 Dec 2014

Dimitris, I'm not into newsfeeds at all, but when I try to create a menu entry for "List News Feeds in Category" I can choose "Uncategorized" only, while I have 2 more categories...

Mind you: this is a fresh install BUT with multilingual already enabled and the other two categories (in which I have articles) are bound to a specific language...

avatar dgt41
dgt41 - comment - 3 Dec 2014

This is for the front end part, what you are describing is the backend. Go to admin->menu create a menu with menu type newsfeed, and visit the front end on that menu. There should be a list of articles with no js errors...

avatar dgt41
dgt41 - comment - 3 Dec 2014

Ooops forgot to tell you that you have to go first to components->newsfeed and set up a feed there!

avatar smanzi
smanzi - comment - 3 Dec 2014

Ah OK... my ignorance... hold-on...

avatar smanzi
smanzi - comment - 3 Dec 2014

Grrrrrrr... I'm not getting how all this works... I even don't see the menu item in my front-end...
Spype + TeamViewer, you come to make me a visit?

avatar smanzi
smanzi - comment - 3 Dec 2014

Thanks @dgt41 for guidance.
@test success

avatar smanzi smanzi - test_item - 3 Dec 2014 - Tested successfully
avatar roland-d roland-d - change - 3 Dec 2014
Status Pending Ready to Commit
avatar roland-d
roland-d - comment - 3 Dec 2014

Moving to RTC as we have 2 successful tests.

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5115.

avatar brianteeman brianteeman - change - 3 Dec 2014
Labels Added: ?
avatar wilsonge
wilsonge - comment - 13 Dec 2014

I can't see what this is being loaded for? If I remove the call to core.js I can't see anything breaking (tested searching and changing pagination). What's this being used for?

avatar dgt41
dgt41 - comment - 13 Dec 2014

@wilsonge try changing the sorting to asc/desc, I guess core.js is required there

avatar wilsonge
wilsonge - comment - 13 Dec 2014

There doesn't seem to be a way to change that?

avatar dgt41
dgt41 - comment - 13 Dec 2014

found it:

<p><?php echo JText::_('COM_NEWSFEEDS_NO_ARTICLES'); ?></p>

JText needs core.js for the translation of the string

avatar wilsonge
wilsonge - comment - 13 Dec 2014

I'm pretty sure the only JText dependency is for using strings in javascript!

avatar dgt41
dgt41 - comment - 13 Dec 2014

@wilsonge You re right, my bad. This is not needed here

avatar zero-24 zero-24 - close - 27 Dec 2014
avatar wilsonge wilsonge - close - 27 Dec 2014
avatar wilsonge wilsonge - change - 27 Dec 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-12-27 18:24:31
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment